RFR: JMC-6211: Restore Defaults button doesn't revert username and password in Preferences

Mario Torre neugens at redhat.com
Tue Mar 5 14:16:05 UTC 2019


On Mon, 2019-03-04 at 19:29 -0500, Elliott Baron wrote:
> 
> Marcus has informed me off-list that the above copyright header
> format 
> is acceptable going forward.
> 
> I do still need a review for the content of this fix. Would someone
> be 
> able to take a look?

Hi Elliott,

The patch looks good to me, I would like to have the opinion of an
official reviewer however before you push it.

A question on the Manifest.

--- a/application/org.openjdk.jmc.console.ui/META-INF/MANIFEST.MF
+++ b/application/org.openjdk.jmc.console.ui/META-INF/MANIFEST.MF

Are those changes in the Manifest really necessary? I take a yes
because the ui tests are in a different package that need internal
visibility?

Cheers,
Mario
-- 
Mario Torre
Associate Manager, Software Engineering
Red Hat GmbH <https://www.redhat.com>
9704 A60C B4BE A8B8 0F30  9205 5D7E 4952 3F65 7898



More information about the jmc-dev mailing list