Community
Participate
Working Groups
Using 3.4M7. Using the following test case: public class X { void foo() { /** * This example will put the star at the beginning of next line: R1 = X1 * 2 */ int b; } } After the format using default Eclipse built-in, I get: public class X { void foo() { /** * This example will put the star at the beginning of next line: R1 = X1 * * 2 */ int b; } } Which looks OK. But if I format this again, I get: public class X { void foo() { /** * This example will put the star at the beginning of next line: R1 = X1 * 2 */ int b; } } The star at the beginning of the line has been wrongly removed!
This must be fixed for 3.4 as some text is removed while formatting!
Created attachment 99260 [details] Proposed patch
With this patch, I now get NO failure on Eclipse 3.0 full source workspace test and 17 on Ganymede workspace test. Olivier, could you please review?
Patch looks good. +1 for 3.4RC1.
Released for 3.4RC1 in HEAD stream.
Verified for 3.4RC1 using build I20080510-2000. Note: found a difference between the "old" and the "new" formatter, but also found that the "new" formatter seems better. old: /** * This example will put the star at the beginning of next line: R1 = X1 * * 2 */ --> the star after X1 is after the 80 margin limit new: /** * This example will put the star at the beginning of next line: R1 = X1 * * 2 */
Verified for 3.4RC1 using build I20080510-2000.
