Details

    • Type: Bug Bug
    • Status: Closed Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 1.12
    • Fix Version/s: 1.12.1, 2.0
    • Component/s: Parser
    • Labels:
      None
    • Testcase included:
      yes
    • Number of attachments :
      1

      Description

      It seems like String arguments to annotations containing escaped backslashes fail.
      @Annotation("This could be a regex
      d")

      public void testTemplate() throws Exception {
      JavaDocBuilder builder = new JavaDocBuilder();
      String source = "public class Foo { @SuppressWarnings(

      {\"a\", \"abc\\d\"}

      )\nprivate void bar() { } }";
      builder.addSource( new StringReader(source) );
      }

        Issue Links

          Activity

          Hide
          P Boon added a comment -

          Exception in thread "main" com.thoughtworks.qdox.parser.ParseException: Illegal escape character 'd' @[1,50] in UNKNOWN SOURCE
          at com.thoughtworks.qdox.parser.impl.Parser.yyerror(Parser.java:1018)
          at com.thoughtworks.qdox.parser.impl.Parser.convertString(Parser.java:1126)
          at com.thoughtworks.qdox.parser.impl.Parser.toString(Parser.java:1233)
          at com.thoughtworks.qdox.parser.impl.Parser.yyparse(Parser.java:1800)
          at com.thoughtworks.qdox.parser.impl.Parser.parse(Parser.java:999)
          at com.thoughtworks.qdox.JavaDocBuilder.addSource(JavaDocBuilder.java:353)
          at com.thoughtworks.qdox.JavaDocBuilder.addSource(JavaDocBuilder.java:343)
          at QDoxIllegalEscapeCharacter.main(QDoxIllegalEscapeCharacter.java:10)

          Show
          P Boon added a comment - Exception in thread "main" com.thoughtworks.qdox.parser.ParseException: Illegal escape character 'd' @ [1,50] in UNKNOWN SOURCE at com.thoughtworks.qdox.parser.impl.Parser.yyerror(Parser.java:1018) at com.thoughtworks.qdox.parser.impl.Parser.convertString(Parser.java:1126) at com.thoughtworks.qdox.parser.impl.Parser.toString(Parser.java:1233) at com.thoughtworks.qdox.parser.impl.Parser.yyparse(Parser.java:1800) at com.thoughtworks.qdox.parser.impl.Parser.parse(Parser.java:999) at com.thoughtworks.qdox.JavaDocBuilder.addSource(JavaDocBuilder.java:353) at com.thoughtworks.qdox.JavaDocBuilder.addSource(JavaDocBuilder.java:343) at QDoxIllegalEscapeCharacter.main(QDoxIllegalEscapeCharacter.java:10)
          Hide
          Robert Scholte added a comment -

          Fixed in rev. 739. I've deployed a 2.0-SNAPSHOT version for you. Be aware that in the final 2.0 version of QDox the model might have changed a bit.

          Show
          Robert Scholte added a comment - Fixed in rev. 739 . I've deployed a 2.0-SNAPSHOT version for you. Be aware that in the final 2.0 version of QDox the model might have changed a bit.
          Robert Scholte made changes -
          Field Original Value New Value
          Status Open [ 1 ] Resolved [ 5 ]
          Assignee Robert Scholte [ rfscholte ]
          Fix Version/s 2.0 [ 15636 ]
          Resolution Fixed [ 1 ]
          Robert Scholte made changes -
          Link This issue is duplicated by QDOX-230 [ QDOX-230 ]
          Robert Scholte made changes -
          Link This issue is depended upon by MPLUGIN-220 [ MPLUGIN-220 ]
          Robert Scholte made changes -
          Resolution Fixed [ 1 ]
          Status Resolved [ 5 ] Reopened [ 4 ]
          Hide
          Robert Scholte added a comment -

          Fixed in rev. 1528 for QDox-1.12.1

          Show
          Robert Scholte added a comment - Fixed in rev. 1528 for QDox-1.12.1
          Robert Scholte made changes -
          Status Reopened [ 4 ] Resolved [ 5 ]
          Fix Version/s 1.12.1 [ 18944 ]
          Resolution Fixed [ 1 ]
          Robert Scholte made changes -
          Status Resolved [ 5 ] Closed [ 6 ]

            People

            • Assignee:
              Robert Scholte
              Reporter:
              P Boon
            • Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: