<div dir="ltr"><div><div><div>Hi !<br><br></div>I try to update the HACKING file considering the need.<br>Feel free to comment and improve :)<br></div><br><a href="https://github.com/geany/geany/pull/224">https://github.com/geany/geany/pull/224</a><br>
<br></div>See you !<br></div><div class="gmail_extra"><br><br><div class="gmail_quote">2014-03-11 10:54 GMT+01:00 Lex Trotman <span dir="ltr"><<a href="mailto:elextr@gmail.com" target="_blank">elextr@gmail.com</a>></span>:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="">On 11 March 2014 19:35, shan chak <<a href="mailto:shankholove@gmail.com">shankholove@gmail.com</a>> wrote:<br>

> Hi guys,<br>
><br>
> My name is Shankhoneer Chakrovarty (aka 'shankhs' ), I have been using geany<br>
> for past 3-4 years (mainly on my ubuntu ) and I have learned all my<br>
> programming skills on geany. So thank you!<br>
><br>
> Since I have been programming for 3-4 years now and also using geany, I<br>
> thought it will be great if I could give something back to the geany<br>
> community.<br>
><br>
> I pulled the geany source code, read the HACKING file and created a patch<br>
> which categorizes the compiler errors into "error" and "warning" and<br>
> correspondingly draws a different colored squiggle underline in the line<br>
> which caused the error. I have created the patch as mentioned in the hacking<br>
> file and attached with this mail.<br>
> If you guys have some time, can you please review the patch? If you need any<br>
> more info, please feel free to drop me an email, I will be more than happy<br>
> to explain it.<br>
<br>
</div>Have not reviewed it in detail, but some comments:<br>
<br>
1. don't change whitespace (eg blank lines, that just makes noise in<br>
the patch that is not needed making real review harder)<br>
2. don't use doxygen comments for in-code comments<br>
3. watch the layout, you are missing many spaces after commas<br>
4. is setting a pointer to a piece of the string and comparing it to<br>
"warning" really the best way of returning that information rather<br>
than a bool or enum?<br>
5. compiler lines are also parsed by filetypes_parse_compiler_line()<br>
<br>
The preferred way of providing changes is now via pull requests on<br>
github rather than patches.  The HACKING needs updating.<br>
<br>
Cheers<br>
Lex<br>
<div class=""><br>
><br>
> Also, I am reading the geany source code and modified some part of it to<br>
> suit my personal needs so if you need a hand in resolving bugs, please do<br>
> tell me. I am particularly interested in<br>
> <a href="https://sourceforge.net/p/geany/bugs/254/" target="_blank">https://sourceforge.net/p/geany/bugs/254/</a> .<br>
<br>
</div>See also #907 :-D<br>
<br>
<br>
><br>
> Thanks,<br>
> Shankhoneer Chakrivarty<br>
><br>
> _______________________________________________<br>
> Devel mailing list<br>
> <a href="mailto:Devel@lists.geany.org">Devel@lists.geany.org</a><br>
> <a href="https://lists.geany.org/cgi-bin/mailman/listinfo/devel" target="_blank">https://lists.geany.org/cgi-bin/mailman/listinfo/devel</a><br>
><br>
_______________________________________________<br>
Devel mailing list<br>
<a href="mailto:Devel@lists.geany.org">Devel@lists.geany.org</a><br>
<a href="https://lists.geany.org/cgi-bin/mailman/listinfo/devel" target="_blank">https://lists.geany.org/cgi-bin/mailman/listinfo/devel</a><br>
</blockquote></div><br></div>