Hi Jure, Thanks for your patches. However, I do have a few comments about the same:
1. Trailing Whitespaces: This is essentially extra whitespaces at the end of a line or on a blank line. See [1] and [2] for more information. 2. The indentation is mostly right, but sometimes off. 3. Your first patch is missing a ChangeLog. Every commit must be accompanied by a ChangeLog entry, no matter how trivial it is. 4. Your 2nd patch seems to revert things from the first one. This usually means some cleanup is needed. I'm not completely sure of some of the details of the lines you change in your second patch, but they seem a little sketchy. I'll have to dig into the code and check it out. Also, for a non-trivial (>10 LoC) patch, you'll first need to submit your copyright assignment to the FSF. Giuseppe will arrange for the documents as soon as your patch is ready. The code however, does fix a segfault and maybe a few compiler warnings. When it fixes something, an explanation is usually a nice idea. [1] http://codeimpossible.com/2012/04/02/Trailing-whitespace-is-evil-Don-t-commit-evil-into-your-repo-/ [2] https://stackoverflow.com/questions/1583406/why-does-git-care-about-trailing-whitespace-in-my-files On Mon, Mar 17, 2014 at 6:56 PM, Jure Grabnar <grabna...@gmail.com> wrote: > Hi, > > this patch fixes some of compiler warnings. I was uncertain for the > remaining ones (5) - I believe some of them might be stubs for upcoming > features. > > Best Regards, > > > Jure Grabnar (toomanysecrets) -- Thanking You, Darshit Shah