-
Notifications
You must be signed in to change notification settings - Fork 223
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
coala-antlr Enhancement #698
Comments
@jayvdb is this suitable for GSOC or should i leave it for non-GSOC work? and better to focus on other areas. |
@Dhiraj240 , I think there are already existing grammars in ANTLR for 8080 and 8086, so it'd be just a kind of an extension to coantlib if you go down that line Also just to mention, I've not come across open source assembly projects except for a few like GRUB (and that too uses intel assembly format, and that too only at the place where it needs to interface during boot time). I think it would be great if you could find some application for this as well, since mostly people just write their source in c, and let the compiler generate / cross-compile applications for whichever platform they need (micro-controllers / pcs etc etc) Moving on to the tree-rewriting part, that'd be an interesting thing to do, but why would you want to do that "without having that in the upstream ANTLR" ? |
I was actually asking that is it needed to implement "tree rewriting at upstream" first ? i guess this work is needed on |
Sure, you can do that only in coala-antlr, but do keep in mind coala-antlr
still uses the core developed by antlr's python port. Unless you plan on
changing all of that and duplicating everything already given by
`antlr4-python3-runtime`, I don't think there would be a lot of trouble.
I see that you did open an issue for tree-rewriting, possibly you should
see the already existing discussions at
antlr/antlr4#369 and follow it up with what have
you thought about it. (Do go through the thread, many people have some good
strategies for this already, and remember that whatever comes to python
port must first satsify the java version's requirement of ANTLR, and not
disrupt the existing architecture, not to mention this has little to do
with coala-antlr if you go down this line)
Also lastly, I didn't say you should work on extending GRUB, that was just
an example of the very rare projects that still use assembly in some part
of their codebase. The main point there was that usually people don't code
in assembly, they simply use a cross-compiler to get what they want from a
much more readable code written in a high level language such as c, and
hardly does anyone care about the readability of the compiled code...
Regards
Viresh Gupta
…On Tue, Feb 5, 2019 at 12:48 AM Dhiraj Sharma ***@***.***> wrote:
I was actually asking that is it needed to implement "tree rewriting at
upstream" first ? i guess this work is needed on coala-antlr only as it
will enhance this project. isn't the upstream ANTLR and coala-antlr
independent now ?
am i right ?
I had serious health issue during the start of new year but revived
though.I will look for GRUB and update more info over it as what all i can
extend to it.
I remember that a basic unittest is still remaining, will be back to work
this week. 😄
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#698 (comment)>, or mute
the thread
<https://github.com/notifications/unsubscribe-auth/AKeDPFRB0Axalb_g_ByWq3owFm1ymt2Bks5vKIeDgaJpZM4ZpUDn>
.
|
Ahh, that "extension" word was for enhancing
What sort of applications ? Can you provide me the example i am unclear about it ? |
From my understanding of embedded C, there is only a very tiny portion of the whole code written in assembly, so in reality, the bigger picture about what you're talking is "nested languages". It's already an existing project for coala, feel free to look it up. Again, "application" means what kind of developers would be interested in using your bear ? Also implementing tree rewriting for only one kind of language would mean you're only supporting source code for one particular language (and thus no work in either coantlib or coala and possibly none in antlr either), in which case it's better to create a separate project for linting such source codes and simply add a bear into coala for your project. (See any existing linter bear). |
Alright, i think i should focus on contribution and learning. |
@virresh alongside to extend this project using asm8085, asm8086 if i add verilog and vhdl too.
Lets keep it basic rather main focus is that is there any way to optimize the Can we use |
Please do reply. |
yes we need this system, and that is what https://gitlab.com/coala/bears/coala-antlr/issues/11 and other related issues are about I would highly advise you to go through the existing issues so that you don't need to spend time on coming up with ideas for things that have been already thought of... I'm not sure what you wish to do with NLTK here, we aren't trying to make sense of words here, we're just stiching together some pieces. Also do read up about which code lines weren't readed, you have commits that can help you determine the same, in case you wish to work on this |
Cool, found a way i will work on proposal based on existing issues and will try NLTK on my code. |
Particularly i will start by adding the grammars which will be asm8080,asm8086 which is intel based processors.For that language definitions for the same will be added in coala.Now the bears for them will be added as of now there are no bears which works for the microprocessors or microcontrollers.Similarly a grammar will be updated on upstream for PIC microcontrollers where a language definition will also be included in coala for the same.Henceforth the bears for the microcontroller and microprocessors will be supported by this project.Basically bears will be used to style the opcode like capitalization of special functions registers etc. which are used for ADC, sensors interfacing.
Apart from this i am planning to make a tree rewriting based on the above grammars generated trees it could be done using the generated template, if it is possible
"without having that functionality on the upstream antlr".
The text was updated successfully, but these errors were encountered: