Error formatting long string literal that contains "...\\t" #1186

Activity

1. freetheinterns commented on Jan 14, 2025

   

   freetheinterns

   on Jan 14, 2025

   Last edited by freetheinterns

   More actions

   I'm also hitting this issue on v1.22. Notably the error also goes away if you manually reflow the string so that it complies with line length limitations.
2. fcorsino commented on Feb 26, 2025

   

   fcorsino

   on Feb 26, 2025

   Last edited by fcorsino

   More actions

   Issue is still happening in 1.25.2

   Seems to have something to do with the length, the backslashes, and case of the characters. Here are some positive and negative cases.

   // This one works
   String test =  "AB\\ABCDEFGH\\ABCDEFGHIJKLMNO\\ABCEFGHIJLMNPQRSTUV\\TRUNK\\SRC\\MAIN\\WEBAPP\\WEB-INF\\ABC";
   // This one does NOT
   String test =  "ab\\abcdefgh\\abcdefghijklmno\\abcefghijlmnpqrstuv\\trunk\\src\\main\\webapp\\web-inf\\abc";
   // This one works
   String test =  "ab\\abcdefgh\\abcdefghijklmno\\abcefghijlmnpqrstuv\\trunk\\src\\main\\webapp\\web-inf\\ab";
   // This one works
   String test =  "12\\12345678\\123456789012345\\1234567890123456789\\12345\\123\\1234\\123456\\1234567\\123";
   
3. copybara-service

   added a commit that references this issue on Oct 30, 2026

   Don't split escaped backslashes when reflowing long string literals.

   ...

   7e4696d
4. copybara-service

   closed this as completedin 7e4696don Oct 30, 2026