Skip to main content

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [List Home]
Re: [recommenders-dev] [soc-dev] [GSoC] Templates weekly report

Sorry, I forgot to add the plan for this week.

What the plan for this week is:

  • Continue testing the editor and the grammar.
    All the above changes contain specific tests, but more are needed to ensure the grammar is correct.
  • Start doing some more research on how to implement code completion, and how to do this for the Template editor.


Best Regards,

Stefan.

P.S. The links from the above mail:

[1]: https://git.eclipse.org/r/#/c/26776/

[2]: https://bugs.eclipse.org/bugs/show_bug.cgi?id=434434

[3]: https://git.eclipse.org/r/#/c/27274/

[4]: https://bugs.eclipse.org/bugs/show_bug.cgi?id=435630

[5]: https://git.eclipse.org/r/#/c/27297/

[6]:https://bugs.eclipse.org/bugs/show_bug.cgi?id=435423

[7]:https://git.eclipse.org/r/#/c/27300/

[8]:https://bugs.eclipse.org/bugs/show_bug.cgi?id=435427




On Wed, May 28, 2014 at 12:01 AM, Stefan Prisca <stefan.prisca@xxxxxxxxx> wrote:
Hi,

This is the weekly report for the Templates project.

First of all, I think the report for last week did not reach the mailing lists. It just occurred to me now that I've somehow managed to send it to myself instead of the lists.
I apologize for this, and I'm forwarding it to you now.

The report for this week is the following:

What the plan for this week was:


  • Merge the changes mentioned above.
  • Further test the editor and add more test cases for the grammar.
    The main focus of this new test cases will be the lexer and the parser. We should be sure that each string is consumed by the rule we expect it to be, and that the grammar is stable enough to move forward.
What has been accomplished:

  • I updated the grammar according to the Java 8 Language Specification.
    A few new separators were introduced ("@", "...", etc) and I've modified the structure of numeric constant terminals.
    Syntax coloring for numeric constants was also fixed by this changes.
    You can track this on gerrit[1] or on bugzilla[2].
  • I pushed a change that helps solve the issue with ${enclosing_type} and ${enclosing_project} template variables.[3]
    These were not declared in the grammar as valid template variables, and were not treated so. You can follow the change here[4]
  • I pushed two more change sets that remove the unnecessary dependencies from the o.e.r.templates.3rd.feature(gerrit: [5], bugzilla: [6]) project and lowered the limits of the dependencies from o.e.r.templates and o.e.r.templates.rcp to XText form 3.0.0 to 2.7.0.(gerrit: [7], bugzilla: [8])


---------- Forwarded message ----------
From: <stefan.prisca@xxxxxxxxx>
Date: Tue, May 20, 2014 at 7:52 PM
Subject: Re: [soc-dev] [GSoC] Templates weekly report
To: "stefan.prisca@xxxxxxxxx" <stefan.prisca@xxxxxxxxx>


​ ​
Hi,

This is the weekly report for the Templates project.

What had been planned for the last week:

  • Resolve the build problem.
  • Write more tests, both for the parser, as well as the lexer. We also need to ensure that the lexer will pass the expected token to the parser. 
    For example, see Bug 434434. Here, the numeric constants might not be matched to the expected tokens, hence there are some highlighting problems. 

What had been done last week:

  • Thanks to Andreas, the build issue from last week has been resolved. Apparently jdt.annotations 2.0.0 was bringing java 8 compiled files. A filter that would ensure a maximum exclusive version of 2.0.0 for jdt.annotations did the trick.
  • I submitted a change set that plans to solve Bug 434434. Except the wrong syntax highlighting, the scientific notation for float numbers was not supported.
    This change set [1] in gerrit adds the support for scientific notations, as well as tests for it and for the way the lexer treats numeric constants.
  • I submitted a change ([2]) that deletes the old snipmatch code from the repository. at [3]
    This change will also add a new feature project (o.e.r.templates.3rd.feature) that will contain the 3rd party dependencies form o.e.r.templates project.
None of the above changes was yet merged. Because the build fails at the moment, I will wait until the change that removes the old snipmatch code and adds the new way of treating 3rd party dependencies is merged, and then merge the others.


What is the plan for the next week:

  • Merge the changes mentioned above.
  • Further test the editor and add more test cases for the grammar.
    The main focus of this new test cases will be the lexer and the parser. We should be sure that each string is consumed by the rule we expect it to be, and that the grammar is stable enough to move forward.

Best Regards,
Stefan.


From: stefan.prisca@xxxxxxxxx
Sent: ‎Tuesday‎, ‎May‎ ‎13‎, ‎2014 ‎9‎:‎44‎ ‎PM
To: soc-dev@xxxxxxxxxxx, recommenders-dev@xxxxxxxxxxx

Hi,

This is the weekly report for the Templates project.

What had been planned for the last week:
  • Continue working on Bug 431623. You can keep track of this on gerrit [1].

  • Add specific tests for more rules. The tests would be similar to the ones described above.
  • Create a feature project for o.e.r.templates. This will make it possible (and simple) for anyone who wishes, to install the editor from the update-site and test it (note that any feedback is highly appreciated 😉 ).
    I will send a notification mail on the lists once the update-site for the editor is up and going.
    The first patch can be found on gerrit [2]. 

What had been done last week:
  • Another patch-set has been pushed to gerrit that solved the conflicts in the grammar. Basically, when there is a conflict between JavaIdentifier and SimpleVariable rules, the parser will choose SimpleVariable.
  • The feature project for o.e.r.templates is working.

A problem with the build occurred before submitting the patch to solve Bug 431623. It appears that one of the dependencies from o.e.r.templates is now bringing o.e.jdt.annotation version 2.0.0 into the build. The classes from this plug-in are compiled with java 8 and are incompatible with our java version.

I’ve been trying to figure out what dependency brings in the o.e.jdt.annotation plug-in, but without any luck. In the dependency tree view from the Eclipse IDE only the 1.1.0 version of  o.e.jdt.annotation is present.

What is the plan for the next week:
  • Resolve the build problem.
  • Write more tests, both for the parser, as well as the lexer. We also need to ensure that the lexer will pass the expected token to the parser. 
    For example, see Bug 434434. Here, the numeric constants might not be matched to the expected tokens, hence there are some highlighting problems. 

Best Regards,
Stefan.

From: stefan.prisca@xxxxxxxxx
Sent: ‎Tuesday‎, ‎May‎ ‎6‎, ‎2014 ‎7‎:‎43‎ ‎PM
To: soc-dev@xxxxxxxxxxx, recommenders-dev@xxxxxxxxxxx

Hi,

This is the weekly report for the Templates project.

What had been planned for the last week:

  • Fix Bug 431623. This will allow the use of java identifiers and keywords as link variable proposals, and also as simple template variables (e.g. ${class}).

What had been done last week:

  • I have submitted a couple of patch-sets that would solve this bug, but it turned out that this causes some conflicts created in the grammar. The problem is that JavaIdentifier and SimpleVariable mostly define the same tokens. A solution would be to make some small restructuring in the grammar and remove any tokens that were defined twice. i.e. SimpleVariable rule will remain as it is and JavaIdentifier will define only the tokens that are not already defined in SimpleVariable. (Maybe some renaming would be good)

  • Added some more test cases that check if the parser matches the input strings with the rules as it is supposed to. This tests are more specific to a certain rule, and make it clearer what rules are matched for certain inputs.

What is the plan for the next week:

  • Continue working on Bug 431623. You can keep track of this on gerrit [1].

  • Add specific tests for more rules. The tests would be similar to the ones described above.
  • Create a feature project for o.e.r.templates. This will make it possible (and simple) for anyone who wishes, to install the editor from the update-site and test it (note that any feedback is highly appreciated 😉 ).
    I will send a notification mail on the lists once the update-site for the editor is up and going.
    The first patch can be found on gerrit [2]. 




Back to the top