Added various Assembly src file-extensions.
Assembly code has a variety of extensions that indicate a particular target system. For instance, ".a51" and ".asm51" contain Assembly code specifically for the 8051 microprocessor. You can view, comment on, or merge this pull request online at:
https://github.com/geany/geany/pull/722
-- Commit Summary --
* Update filetype_extensions.conf
-- File Changes --
M data/filetype_extensions.conf (2)
-- Patch Links --
https://github.com/geany/geany/pull/722.patch https://github.com/geany/geany/pull/722.diff
--- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/pull/722
@@ -7,7 +7,7 @@ Abc=*.abc;*.abp; ActionScript=*.as; Ada=*.adb;*.ads; Asciidoc=*.asciidoc;*.adoc; -ASM=*.asm; +ASM=*.asm;*.asm51;*.a51;*.asm;*.ll;*.s03;*.s79;*.s82;*.s90;*.s;
`*.asm` is duplicated. I can't find anything using `*.ll` as an assembly extension. Seems to be use by [Lex](https://en.wikipedia.org/wiki/Lex_%28software%29) and some obscure web stuff or something, but nothing about assembly.
`*.S` might be a nice addition, too.
--- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/pull/722/files#r43577034
Does all those extensions refer to assembly dialects the ASM filetype actually supports? There is like a gazillion dialects, and they often are completely different, so it needs to actually be compatible.
--- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/pull/722#issuecomment-152779440
@@ -7,7 +7,7 @@ Abc=*.abc;*.abp; ActionScript=*.as; Ada=*.adb;*.ads; Asciidoc=*.asciidoc;*.adoc; -ASM=*.asm; +ASM=*.asm;*.asm51;*.a51;*.asm;*.ll;*.s03;*.s79;*.s82;*.s90;*.s;
Whoops. I bet that duplicate mistake does not give me a good first-impression on this team. (^u^)
*.ll is LLVM assembly.
From now on, I will be discuss my intended commits and ideas on the Geany-dev mailing list before submitting contributions. Hopefully, that will save Geany's GitHub team some time and inconvenience. I will also include more detailed commit notes and be more detailed in explanations.
--- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/pull/722/files#r43583505
@b4n , true, there are numerous assembly extensions. Geany does support 8051 assembly, so I added those extensions (asm51 & a51) since I use them sometimes.
--- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/pull/722#issuecomment-152834025
The lexer supports a sort of combined NASM/MASM assembler format (according to the comment) and doesn't care what the target chip is so it should be fairly flexible.
--- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/pull/722#issuecomment-152877569
Closed #722.
--- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/pull/722#event-461382789
github-comments@lists.geany.org