[Geany-Devel] Hi!

[Shankhoneer Chakrovarty]

Thanks Lex. Sorry for the late reply.

You seemed to be returning "is this a warning or not", so thats a
> boolean condition that can be tested locally and returned as such.
> Returning a pointer to part of the message and then testing it in code
> far far away is likely to be confusing for maintenance.


I agree and I understand your concern. The problem I have is if I modify
the functions (msgwin_parse_compiler_error_line and others in the chain) to
return a boolean value(whether its an error or warning) I have no way to
test whether the function continues to work for all the  filetypes
supported by geany. Functions like msgwin_parse_compiler_error_line and
others are extremely generic and is supposed to work for all the filetypes.
I have tested my code for C, C++, Java, Python and LaTex and it seems to
work. This brings me to my second point, can you please share with me any
testing code that you guys might have?


> Then it would be best to make a pull request on github so we can
> attach comments to the source, thanks.


I have made the pull request. Again thank you for being so patient and
helping me understand all the nitty-gritties of geany source code.

Thanks,
Shankhoneer Chakrovarty
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.geany.org/pipermail/devel/attachments/20140316/d325ab32/attachment.html>