Brian,

There were 2 comments of mine on the PR but you only replied to one. If you could reply to the second one, I'll be able review it and approve it ;-) 

Thanks 
Best regards,
/Xavier



On 04 Feb 2014, at 15:01, Brian Fitzpatrick <notifications@github.com> wrote:

> In plugins/org.jboss.tools.ws.core/src/org/jboss/tools/ws/core/command/UninstallJBossWSClassPathCommand.java:
> 
> > +			if (classpathEntries.size() < oldClasspathEntries.length) {
> > +				javaProject.setRawClasspath(classpathEntries
> > +						.toArray(new IClasspathEntry[classpathEntries.size()]),
> > +						new NullProgressMonitor());
> > +			}
> > +		} catch (JavaModelException e) {
> > +			status = StatusUtils.errorStatus(NLS.bind(
> > +					JBossWSCoreMessages.Error_Remove_Facet_JBossWS,
> > +					new String[] { e.getLocalizedMessage() }), e);
> > +			return status;
> > +		}
> > +
> > +		return status;
> > +	}
> > +
> > +}
> I really can't (don't know how) - this is how the Git tooling shoved those changes in - but it's just those commented lines #43 to #56, changing the "if (isServerSupplied)" clause to "if (!isServerSupplied)" - nothing fancy.
> 
> —
> Reply to this email directly or view it on GitHub.
>