UIMA
  1. UIMA
  2. UIMA-510

JCasGen uses an older Java model for merging hand-coded code with generated code, which doesn't support Java beyond the 1.4 level.

    Details

    • Type: Bug Bug
    • Status: Open
    • Priority: Major Major
    • Resolution: Unresolved
    • Affects Version/s: 2.1, 2.2, 2.3
    • Fix Version/s: None
    • Component/s: Tools
    • Labels:
      None

      Description

      JCasGen has a "merge" functionality to merge user-written code in previous versions of the generated JCas cover classes, with regenerated versions of these cover classes, so as to preserve the user-written code (new code or modifications). The functionality is provided by EMF. EMF started issuing warning messages that the Java modelling package it uses, "JDOM", was not updated for Java versions beyond Java 1.4. Because of this, running JCasGen gives the following message in the Eclipse Error Log: Using the JDOM API when the source compatibility is not set to '1.4' or lower can cause unpredictable results.

      The fix is to see if our use of EMF for this can be modified to use JDT's AST APIs instead. For EMF itself, there is a property for the Facade Helper Class in the GenModel that can be switched to org.eclipse.emf.codegen.merge.java.facade.ast.ASTFacadeHelper - this might be where to start looking.

        Activity

        Hide
        Marshall Schor added a comment -

        Investigation shows that the EMF merge code was updated to handle Java 1.5 (and later?) as of EMF 2.2, which corresponds to Eclipse 3.2. How serious a problem is it if we have
        Eclipse 3.2 as the minimum level of Eclipse we run with?

        Show
        Marshall Schor added a comment - Investigation shows that the EMF merge code was updated to handle Java 1.5 (and later?) as of EMF 2.2, which corresponds to Eclipse 3.2. How serious a problem is it if we have Eclipse 3.2 as the minimum level of Eclipse we run with?
        Hide
        Tommaso Teofili added a comment -

        +1

        Show
        Tommaso Teofili added a comment - +1

          People

          • Assignee:
            Unassigned
            Reporter:
            Marshall Schor
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:

              Development