LGTM

This also partialy fixes bug 13850. While switching to the publisher with exprider certificate, the user now doesn't end up having unusable PM. The error dialog is shown and PM closes when clicking on the Close button.


Michal


On 01/14/10 10:57 AM, Padraig O'Briain wrote:
This looks good to me.

Padraig

On 01/13/10 15:28, John Rice wrote:
Sorry posted work in progress webrev, correct pylinted one is at:
http://cr.opensolaris.org/~jmr/pm_13852_global_error_handler_13Jan_325pm/

JR

John Rice wrote:
Hi - this webrev adds a global error handler to PM to catch any unhandled exceptions and display them to the user.

http://cr.opensolaris.org/~jmr/pm_13852_global_error_handler_13Jan_320pm/ Code review: 13852 PM needs a global error handler for uncaught exceptions

I also intend to log caught ApiExceptions and make the Logs available from a View->Log menu, but that's for another webrev.

JR
_______________________________________________
pkg-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/pkg-discuss

_______________________________________________
pkg-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/pkg-discuss
_______________________________________________
pkg-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/pkg-discuss

_______________________________________________
pkg-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/pkg-discuss

Reply via email to