Mike Spivey

Two comments on the proposed fix: (1) There is support only for a single comment character when, as someone noted earlier, at least one assembler dialect uses "//". It's not hard to allow a multi-character string, which is what I did in patching Geany. If a string is allowed (and perhaps in any case) it would be better to call the property lexer.as.comment.prefix instead of lexer.as.comment.character. (2) The fix does not allow /*... */ comments, which are permitted in GNU as. Actually, the infrestructure...

Two comments on the proposed fix: (1) There is support only for a single comment character when, as someone noted earlier, at least one assembler dialect uses "//". It's not hard to allow a multi-character string, which is what I did in patching Geany. If a string is allowed (and perhaps in any case) it would be better to call the property lexer.as.comment.prefix instead of lexer.as.comment.character. (2) The fix does not allow /*... */ comments, which are permitted in GNU as. Actually, the infrestructure...

Two comments on the proposed fix: (1) There is support only for a single comment character when, as someone noted earlier, at least one assembler dialect uses "//". It's not hard to allow a multi-character string, which is what I did in patching Geany. If a string is allowed (and perhaps in any case) it would be better to call the property lexer.as.comment.prefix instead of lexer.as.comment.character. (2) The fix does not allow / ... / comments, which are permitted in GNU as. Actually, the infrestructure...

Was there any progress with the main issue -- making the lexer LexAsm.cxx adaptable to different comment conventions? It's not hard to make the comment character a configuration option, and to make C-style comments an option too. I will prepare a patch if there is any appetite to include it. The current lexer (and the list of instructions that appear in Geany's config file) seems not to suit any particular machine or assembler. The best thing, I think, would be to follow closely the machine-independent...