Description Frederic Fusier 2003-10-10 10:24:30 EDT

I noticed that in certain circumstances (not always) the NON-NLS comment is not 
kept on the same line.
Here is the sample on which it happens for me:
		if (JavaModelManager.CP_RESOLVE_VERBOSE) {
			System.out.println("CPContainer SET  - setting 
container: [" //$NON-NLS-1$
					+ containerPath
					+ "] for projects: {" //$NON-NLS-1$
					+ 
(org.eclipse.jdt.internal.compiler.util.Util.toString(affectedProjects,
							new 
org.eclipse.jdt.internal.compiler.util.Util.Displayable() {
								public String 
displayString(Object o) {
									return 
((IJavaProject) o).getElementName();
								}
							}))
					+ "} with values: " //$NON-NLS-1$
					+ 
(org.eclipse.jdt.internal.compiler.util.Util.toString(respectiveContainers,
							new 
org.eclipse.jdt.internal.compiler.util.Util.Displayable() {
								public String 
displayString(Object o) {
									return 
((IClasspathContainer) o).getDescription();
								}
							})));
		}

then after formatting, I got:
		
		if (JavaModelManager.CP_RESOLVE_VERBOSE) {
			System.out.println("CPContainer SET  - setting 
container: ["
					//$NON-NLS-1$
					+ containerPath
					+ "] for projects: {"
					//$NON-NLS-1$
					+ 
(org.eclipse.jdt.internal.compiler.util.Util.toString(affectedProjects,
							new 
org.eclipse.jdt.internal.compiler.util.Util.Displayable() {
								public String 
displayString(Object o) {
									return 
((IJavaProject) o).getElementName();
								}
							}))
					+ "} with values: "
					//$NON-NLS-1$
					+ 
(org.eclipse.jdt.internal.compiler.util.Util.toString(respectiveContainers,
							new 
org.eclipse.jdt.internal.compiler.util.Util.Displayable() {
								public String 
displayString(Object o) {
									return 
((IClasspathContainer) o).getDescription();
								}
							})));
		}

It's a little bit tricky to see with these line breaks put by bugzilla, but 
I've already talked about this with Olivier and sent him precise reference...