[Bug 1743] Intermittant deadlock in PluginRequestProcessor

[bugzilla-daemon at icedtea.classpath.org]

http://icedtea.classpath.org/bugzilla/show_bug.cgi?id=1743

Kurt Miller <kurt at intricatesoftware.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Attachment #1073|0                           |1
        is obsolete|                            |

--- Comment #2 from Kurt Miller <kurt at intricatesoftware.com> ---
Created attachment 1085
  --> http://icedtea.classpath.org/bugzilla/attachment.cgi?id=1085&action=edit
Deadlock and global variable fix -version 2

Thanks for reviewing and testing.

I've attached an updated patch which does the following:
. makes the message_queue and related variables private
. added a new public method called queueProcessorThread() where the logic that
was in queue_processor() is moved to queueProcessorThread()
. I noted that several public methods called by queueProcessorThread() are not
used outside the class anymore so I privatized these methods as well. 

This patch should be no functional change when compared to version 1.
PluginRequestProcessor public interface is very simple now; constructor,
destructor, queing method and dequeing thread method.

Thanks again for your review. Please let me know if there are any questions or
changes needed.

Regards,
-Kurt

-- 
You are receiving this mail because:
You are on the CC list for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openjdk.java.net/pipermail/distro-pkg-dev/attachments/20140521/c9cc542e/attachment.html>