refactor: reduce SuppressWarnings scope; use Thread#sleep

Author: oldratlee

src/main/java/com/oldratlee/fucking/concurrency/HashMapHangDemo.java

@@ -8,10 +8,10 @@
  * @author Jerry Lee (oldratlee at gmail dot com)
  * @see <a href="http://coolshell.cn/articles/9606.html">Infinite loop of Java HashMap</a> by <a href="http://github.com/haoel">@haoel</a>
  */
-@SuppressWarnings("InfiniteLoopStatement")
 public class HashMapHangDemo {
     final Map<Integer, Object> holder = new HashMap<>();
 
+    @SuppressWarnings("InfiniteLoopStatement")
     public static void main(String[] args) {
         HashMapHangDemo demo = new HashMapHangDemo();
         for (int i = 0; i < 100; i++) {
@@ -43,6 +43,7 @@ private class ConcurrencyTask implements Runnable {
         private final Random random = new Random();
 
         @Override
+        @SuppressWarnings("InfiniteLoopStatement")
         public void run() {
             System.out.println("Add loop started in task!");
             while (true) {

src/main/java/com/oldratlee/fucking/concurrency/InvalidLongDemo.java

@@ -3,10 +3,10 @@
 /**
  * @author Jerry Lee(oldratlee at gmail dot com)
  */
-@SuppressWarnings("InfiniteLoopStatement")
 public class InvalidLongDemo {
     long count = 0;
 
+    @SuppressWarnings("InfiniteLoopStatement")
     public static void main(String[] args) {
         // LoadMaker.makeLoad();
 
@@ -28,6 +28,7 @@ ConcurrencyCheckTask getConcurrencyCheckTask() {
 
     private class ConcurrencyCheckTask implements Runnable {
         @Override
+        @SuppressWarnings("InfiniteLoopStatement")
         public void run() {
             int c = 0;
             for (int i = 0; ; i++) {

src/main/java/com/oldratlee/fucking/concurrency/NoPublishDemo.java

@@ -1,7 +1,5 @@
 package com.oldratlee.fucking.concurrency;
 
-import com.oldratlee.fucking.concurrency.util.Utils;
-
 /**
  * <a href="https://hllvm-group.iteye.com/group/topic/34932">请问R大 有没有什么工具可以查看正在运行的类的c/汇编代码</a>提到了<b>代码提升</b>的问题。
  *
@@ -10,15 +8,15 @@
 public class NoPublishDemo {
     boolean stop = false;
 
-    public static void main(String[] args) {
+    public static void main(String[] args) throws Exception {
         // LoadMaker.makeLoad();
 
         NoPublishDemo demo = new NoPublishDemo();
 
         Thread thread = new Thread(demo.getConcurrencyCheckTask());
         thread.start();
 
-        Utils.sleep(1000);
+        Thread.sleep(1000);
         System.out.println("Set stop to true in main!");
         demo.stop = true;
         System.out.println("Exit main.");
@@ -28,15 +26,15 @@ ConcurrencyCheckTask getConcurrencyCheckTask() {
         return new ConcurrencyCheckTask();
     }
 
-    @SuppressWarnings({"StatementWithEmptyBody", "WhileLoopSpinsOnField"})
     private class ConcurrencyCheckTask implements Runnable {
         @Override
+        @SuppressWarnings({"WhileLoopSpinsOnField", "StatementWithEmptyBody"})
         public void run() {
             System.out.println("ConcurrencyCheckTask started!");
             // If the value of stop is visible in the main thread, the loop will exit.
             // On my dev machine, the loop almost never exits!
             // Simple and safe solution:
-            //   add volatile to the running field.
+            //   add volatile to the `stop` field.
             while (!stop) {
             }
             System.out.println("ConcurrencyCheckTask stopped!");

src/main/java/com/oldratlee/fucking/concurrency/util/LoadMaker.java

@@ -33,7 +33,7 @@ public static void stopLoad() {
         isLoadMade = false;
     }
 
-    public static void sleep(long millis) {
+    private static void sleep(long millis) {
         try {
             Thread.sleep(millis);
         } catch (InterruptedException ignored) {