<div dir="ltr">Thanks Lex. Please see my comments and questions inline.<div class="gmail_extra"><br><div class="gmail_quote">On Tue, Mar 11, 2014 at 2:54 AM, Lex Trotman <span dir="ltr"><<a href="mailto:elextr@gmail.com" target="_blank">elextr@gmail.com</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style: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)</blockquote><div>[Shankhoneer] Point noted. I will remove these noises in my patch </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">

 </blockquote><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
2. don't use doxygen comments for in-code comments<br></blockquote><div>[Shankhoneer] I have changed all in-code doxygen comments to c-style comments </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">


3. watch the layout, you are missing many spaces after commas<br></blockquote><div>[Shankhoneer] Fixed this too </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">


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></blockquote><div>[Shankhoneer] I dont understand exactly what you are trying to suggest. Do you mean that I should create a function that returns which color the line should be given the error message? I have never used C before in production software as big as geany so please feel free to comment directly what would be a better approach.</div>

<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
5. compiler lines are also parsed by filetypes_parse_compiler_line()<br></blockquote><div>[Shankhoneer] I couldnt find filetypes_parse_compiler_line() but I found filetypes_parse_error_message() in filetypes.c</div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">


<br>
The preferred way of providing changes is now via pull requests on<br>
github rather than patches.  The HACKING needs updating.<br></blockquote><div>[Shankhoneer] I have forked the project and working on the forked repo. </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">

<div class="">><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></blockquote><div>[Shankhoneer] Woah! #907 is like a long TODO list for C++11. I have started to look into some of the issues like 5e. </div><div><br></div><div>Thanks,</div><div>Shankhoneer Chakrovarty </div>

</div></div></div>