[please respond inline rather than top-posting] On Fri, Mar 25, 2016 at 8:53 AM, work <[email protected]> wrote: > On 03/25/2016 09:18 AM, Junio C Hamano wrote: >> Stefan Beller <[email protected]> writes: >>> Maybe my exposure to the code was accidentally in a way such that >>> I ever only saw the version without int. >> >> The older part of the code tends to spell flag words with "unsigned" >> without "int", which is primarily historical fault of mine. > > Yep. Thanks for your remarks. I have made a bit more research about using > old rev_list_info struct (with signed int flag) and realized, that it > doesn't appear in expressions, where using signed integer will differ from > unsigned one. > I'll take using 'unsigned' instead of 'unsigned int' in account, so if > needed, I can remake the patch in order to get it accepted.
If I read Junio's response correctly, "unsigned int" is indeed preferred over "unsigned", so no need to change that part, but the commit message needs improvement, and other reviewer comments should be addressed. And, yes, the expectation is that you will re-roll the patch (one or more times) in response to issues pointed out by reviewers. As a GSoC applicant, this is especially important since a big part of working on this project is being responsive to review comments, and GSoC mentors will examine your reviewer interaction when selecting applicants. -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
