Syntax Highlighting for AspectJ

[ozlerhakan]

I tackled the AspectJ language in order to make its keywords more readeable for all user.
I hope this pull will be accepted by you.

[ozlerhakan]

could you please explain why this should be detected as java. AspectJ has a separate source code file ending with .aj and it supports nested interface, class and aspect declarations.

[isagalaev]

The wording of the test is not very clear, sorry. The problem is that with the addition of your AspectJ definition the Java code is now being mis-detected as AspectJ. A way to fix this is to build the browser package with only those two languages and look at test.html to see what AspectJ finds in Java snippet that it shouldn't. It's tricky though :-).

However there are a few simpler things to before diving into cross-relevance magic:

• Please re-indent the code with two space for consistency with the rest of the library.
• The test snippet should be way, way smaller. The ideal size is just a few lines showing important aspects of the language, it shouldn't be a big chunk of real-world code. Look at Python or C++ for example.
• You've defined quite a bit of language specific class names ("shortkeys", "aspect", "associations", etc.) but they simply won't be visibly highlighted without styles. Stick to the common class names. In fact, we're going to be more strict about this policy in the future. It's perfectly okay to define parsing modes without class names, especially if they're unique for the language, as they help in language detection.

[ozlerhakan]

Thank you very much I will consider all of them :)