Changeset 101 for java/main/src/main/java/com/framsticks/util/dispatching/Thread.java

Timestamp:

07/14/13 23:20:04 (11 years ago)

Author:

psniegowski

Message:

HIGHLIGHTS:

• improve tree side notes
• improve GUI layout
• add foldable list of occured events to EventControl?
• improve automatic type conversion in proxy listeners
• implement several Access functionalities as algorithms independent of Access type
• introduce draft base classes for distributed experiments
• automatically register dependant Java classes to FramsClass? registry
• add testing prime experiment and configuration
• simplify and improve task dispatching

CHANGELOG:
Improve task dispatching in RemoteTree?.

GUI no longer hangs on connection problems.

Make all dispatchers joinables.

Refactorize Thread dispatcher.

Remove Task and PeriodicTask?.

Use Java utilities in those situations.

Reworking tasks dispatching.

Fix bug in EventControl? listener dispatching.

Minor improvements.

Add testing configuration for ExternalProcess? in GUI.

More improvement to prime.

Support for USERREADONLY in GUI.

Add that flag to various params in Java classes.

Remove redundant register clauses from several FramsClassAnnotations?.

Automatically gather and register dependant classes.

Add configuration for prime.

Improve Simulator class.

Add prime.xml configuration.

Introduce draft Experiment and Simulator classes.

Add prime experiment tests.

Enclose typical map with listeners into SimpleUniqueList?.

Needfile works in GUI.

Improve needfile handling in Browser.

More improvement with NeedFile?.

Implementing needfile.

Update test.

Rename ChangeEvent? to TestChangeEvent?.

Automatic argument type search in RemoteTree? listeners.

MultiParamLoader? uses AccessProvider?. By default old implementation
enclosed in AccessStash? or Registry.

Minor changes.

Rename SourceInterface? to Source.

Also improve toString of File and ListSource?.

Remove unused SimpleSource? class.

Add clearing in HistoryControl?.

Show entries in table at EventControl?.

Improve EventControl?.

Add listeners registration to EventControl?.

Add foldable table to HistoryControl?.

Add control row to Procedure and Event controls.

Improve layout of controls.

Another minor change to gui layout.

Minor improvement in the SliderControl?.

Minor changes.

Move ReflectionAccess?.Backend to separate file.

It was to cluttered.

Cleanup in ReflectionAccess?.

Move setMin, setMax, setDef to AccessOperations?.

Extract loading operation into AccessOperations?.

Append Framsticks to name of UnsupportedOperationException?.

The java.lang.UnsupportedOperationException? was shadowing this class.

Rename params.Util to params.ParamsUtil?.

Several improvements.

• Merge ValueControlListener? into ControlOwner?.
• Remove several deprecated utility classes.
• Add EventListeners? utility container.

Minor changes.

Implement revert functionality.

Improve local changes management.

Minor improvement.

Remove methods rendered superfluous after SideNoteKey? improvement.

Improve SideNoteKey?.

It is now generic type, so explicit type specification at
call site is no more needed.

Introduce SideNoteKey? interface.

Only Objects implementing that key may be used as side note keys.

Minor improvements.

Use strings instead of ValueControls? in several gui mappings.

File:

• java/main/src/main/java/com/framsticks/util/dispatching/Thread.java (modified) (6 diffs)

java/main/src/main/java/com/framsticks/util/dispatching/Thread.java

@@ -3 +3 @@
 import org.apache.logging.log4j.Logger;
 import org.apache.logging.log4j.LogManager;
-
-import java.util.LinkedList;
-import java.util.ListIterator;
 
 
@@ -14 +11 @@
  * @author Piotr Sniegowski
  */
-public class Thread<C> extends AbstractJoinable implements JoinableDispatcher<C> {
+public class Thread<C> extends AbstractJoinable implements Dispatcher<C> {
 
         private static final Logger log = LogManager.getLogger(Thread.class);
@@ -20 +17 @@
         protected final java.lang.Thread thread;
 
-        private final LinkedList<Task<? extends C>> queue = new LinkedList<>();
+        protected final Object condition = new Object();
+        private RunnableQueue<C> queue = new RunnableQueue<>();
 
         public Thread() {
-                thread = new java.lang.Thread(new java.lang.Runnable() {
+                thread = new java.lang.Thread(new Runnable() {
                         @Override
                         public void run() {
@@ -30 +28 @@
                 });
         }
-
 
         public Thread(java.lang.Thread thread) {
@@ -51 +48 @@
                 ExceptionHandler exceptionHandler = getMonitor().getTaskExceptionHandler();
                 while (!java.lang.Thread.interrupted()) {
-                        Task<? extends C> task;
-                        synchronized (queue) {
+                        RunAt<? extends C> runnable;
+                        synchronized (condition) {
                                 if (queue.isEmpty()) {
                                         try {
-                                                queue.wait();
+                                                condition.wait();
                                         } catch (InterruptedException ignored) {
                                                 break;
@@ -61 +58 @@
                                         continue;
                                 }
-                                task = queue.peekFirst();
-                                assert task != null;
-                                if (task.moment > System.currentTimeMillis()) {
-                                        try {
-                                                queue.wait(task.moment - System.currentTimeMillis());
-                                        } catch (InterruptedException ignored) {
-                                                continue;
+                                runnable = queue.pollFirst();
+                        }
+                        if (runnable != null) {
+                                try {
+                                        runnable.run();
+                                } catch (Exception e) {
+                                        if (exceptionHandler != null) {
+                                                if (exceptionHandler.handle(this, e)) {
+                                                        continue;
+                                                }
                                         }
-                                        continue;
+                                        log.error("error in thread: ", e);
                                 }
-                                queue.pollFirst();
-                        }
-                        try {
-                                task.run();
-                        } catch (Exception e) {
-                                if (exceptionHandler != null) {
-                                        if (exceptionHandler.handle(this, e)) {
-                                                continue;
-                                        }
-                                }
-                                log.error("error in thread: ", e);
                         }
                 }
                 log.debug("finishing thread {}", this);
-                finish();
+                finishJoinable();
         }
 
-        protected void enqueueTask(Task<? extends C> task) {
-                synchronized (queue) {
-                        ListIterator<Task<? extends C>> i = queue.listIterator();
-                        while (i.hasNext()) {
-                                Task<? extends C> t = i.next();
-                                if (t.getMoment() > task.getMoment()) {
-                                        i.previous();
-                                        i.add(task);
-                                        task = null;
-                                        break;
-                                }
-                        }
-                        if (task != null) {
-                                queue.add(task);
-                        }
 
-                        /*
-                        Iterator<Task> j = queue.iterator();
-                        Task prev = null;
-                        while (j.hasNext()) {
-                                Task next = j.next();
-                                assert (prev == null) || prev.getMoment() <= next.getMoment();
-                                prev = next;
-                        }
-                        */
-                        queue.notify();
+        @Override
+        public void dispatch(RunAt<? extends C> runnable) {
+                synchronized (condition) {
+                        queue.push(runnable);
+                        condition.notifyAll();
                 }
         }
 
-        @Override
-        public void dispatch(final RunAt<? extends C> runnable) {
-                if (!(runnable instanceof Task)) {
-                        enqueueTask(new Task<C>(runnable) {
-                                @Override
-                                protected void runAt() {
-                                        runnable.run();
-                                }
-                        });
-                        return;
+        public RunnableQueue<C> switchQueue(RunnableQueue<C> queue) {
+                synchronized (condition) {
+                        RunnableQueue<C> result = this.queue;
+                        this.queue = queue;
+                        return result;
                 }
-                enqueueTask((Task<? extends C>) runnable);
         }