{"thread":{"id":"47693","subject":"Please review my code","startedAt":"2018-01-25T17:20:47Z","lastAt":"2018-01-26T19:34:55Z","messageCount":5,"participants":["Оля Тележная","Christian Couder"],"isPatch":false,"patchVersion":null,"patchTotal":null},"messages":[{"id":"337389","messageId":"CAL21BmkTbr9qYK3+bsbwh9aDQ1twqrNkyUdbaFPSAn5y7ov40Q@mail.gmail.com","threadId":"47693","inReplyTo":null,"subject":"Please review my code","fromName":"Оля Тележная","fromEmail":"olyatelezhnaya@gmail.com","sentAt":"2018-01-25T17:20:41Z","receivedAt":"2018-01-25T17:20:47Z","isPatch":false,"sender":{"key":"olyatelezhnaya@gmail.com","avatar":"https://avatars.githubusercontent.com/u/11246099?v=4"},"body":"Hi everyone,\nI haven't sent the code by mailing lists because 25 commits (every\ncommit in separate message) look like a spam.\n\nPlease look at my code:\nhttps://github.com/telezhnaya/git/commits/catfile\nYou could send me any ideas here or in Github.\n\nThe main idea of the patch is to get rid of using custom formatting in\ncat-file and start using general one from ref-filter.\nAdditional bonus is that cat-file becomes to support many new\nformatting commands like %(if), %(color), %(committername) etc.\n\nI remember I need to rewrite docs, I will do that in the near future.\n\nI would be happy to get any ideas from you.\nThanks!\nOlga\n"},{"id":"337403","messageId":"CAP8UFD14V-3T0kV=VxYLdozGp5yT-Rzio7PiY8sFn_rVUDxnjA@mail.gmail.com","threadId":"47693","inReplyTo":"CAL21BmkTbr9qYK3+bsbwh9aDQ1twqrNkyUdbaFPSAn5y7ov40Q@mail.gmail.com","subject":"Re: Please review my code","fromName":"Christian Couder","fromEmail":"christian.couder@gmail.com","sentAt":"2018-01-25T20:22:53Z","receivedAt":"2018-01-25T20:24:02Z","isPatch":false,"sender":{"key":"christian.couder@gmail.com","avatar":"https://avatars.githubusercontent.com/u/208954?v=4"},"body":"Hi Olga,\n\nOn Thu, Jan 25, 2018 at 6:20 PM, Оля Тележная wrote:\n> Hi everyone,\n> I haven't sent the code by mailing lists because 25 commits (every\n> commit in separate message) look like a spam.\n\nYeah, so now that you added tests, it might be interesting to see if\nthe patch series can be refactored to be shorter or to be clearer.\n\n> Please look at my code:\n> https://github.com/telezhnaya/git/commits/catfile\n> You could send me any ideas here or in Github.\n\nI left some comments on GitHub. My main suggestion is to try to get\nrid of the is_cat global and if possible to remove the \"struct\nexpand_data *cat_file_info\" global.\n\n> The main idea of the patch is to get rid of using custom formatting in\n> cat-file and start using general one from ref-filter.\n> Additional bonus is that cat-file becomes to support many new\n> formatting commands like %(if), %(color), %(committername) etc.\n\nYeah, that is a really nice result.\n\n> I remember I need to rewrite docs, I will do that in the near future.\n\nGreat, thanks,\nChristian.\n"},{"id":"337458","messageId":"CAL21Bm=h4o5k2mQ7vob3m-6N-YZ4Rmr5X3_w90ifpR53_+Wnyw@mail.gmail.com","threadId":"47693","inReplyTo":"CAP8UFD14V-3T0kV=VxYLdozGp5yT-Rzio7PiY8sFn_rVUDxnjA@mail.gmail.com","subject":"Re: Please review my code","fromName":"Оля Тележная","fromEmail":"olyatelezhnaya@gmail.com","sentAt":"2018-01-26T10:32:13Z","receivedAt":"2018-01-26T10:32:20Z","isPatch":false,"sender":{"key":"olyatelezhnaya@gmail.com","avatar":"https://avatars.githubusercontent.com/u/11246099?v=4"},"body":"2018-01-25 23:22 GMT+03:00 Christian Couder :\n> Hi Olga,\n>\n> On Thu, Jan 25, 2018 at 6:20 PM, Оля Тележная wrote:\n>> Hi everyone,\n>> I haven't sent the code by mailing lists because 25 commits (every\n>> commit in separate message) look like a spam.\n>\n> Yeah, so now that you added tests, it might be interesting to see if\n> the patch series can be refactored to be shorter or to be clearer.\n>\n>> Please look at my code:\n>> https://github.com/telezhnaya/git/commits/catfile\n>> You could send me any ideas here or in Github.\n>\n> I left some comments on GitHub. My main suggestion is to try to get\n> rid of the is_cat global and if possible to remove the \"struct\n> expand_data *cat_file_info\" global.\n\nThanks for your comments, I find them very useful. Most of issues are\nfixed except the main one, that you mentioned here :)\nI left the comment on GitHub.\nhttps://github.com/telezhnaya/git/commit/403ab584fbf543acef1ecdf279ce31f4fc2ced3f\n\nThanks again!\nOlga\n\n>\n>> The main idea of the patch is to get rid of using custom formatting in\n>> cat-file and start using general one from ref-filter.\n>> Additional bonus is that cat-file becomes to support many new\n>> formatting commands like %(if), %(color), %(committername) etc.\n>\n> Yeah, that is a really nice result.\n>\n>> I remember I need to rewrite docs, I will do that in the near future.\n>\n> Great, thanks,\n> Christian.\n"},{"id":"337487","messageId":"CAP8UFD31oUtoPMN+S_r5YoKrSN4p_QgZQPE6dF20Wdf6o6vdJw@mail.gmail.com","threadId":"47693","inReplyTo":"CAL21Bm=h4o5k2mQ7vob3m-6N-YZ4Rmr5X3_w90ifpR53_+Wnyw@mail.gmail.com","subject":"Re: Please review my code","fromName":"Christian Couder","fromEmail":"christian.couder@gmail.com","sentAt":"2018-01-26T16:42:48Z","receivedAt":"2018-01-26T16:42:54Z","isPatch":false,"sender":{"key":"christian.couder@gmail.com","avatar":"https://avatars.githubusercontent.com/u/208954?v=4"},"body":"On Fri, Jan 26, 2018 at 11:32 AM, Оля Тележная wrote:\n> 2018-01-25 23:22 GMT+03:00 Christian Couder :\n>> On Thu, Jan 25, 2018 at 6:20 PM, Оля Тележная wrote:\n>>> Please look at my code:\n>>> https://github.com/telezhnaya/git/commits/catfile\n>>> You could send me any ideas here or in Github.\n>>\n>> I left some comments on GitHub. My main suggestion is to try to get\n>> rid of the is_cat global and if possible to remove the \"struct\n>> expand_data *cat_file_info\" global.\n>\n> Thanks for your comments, I find them very useful. Most of issues are\n> fixed except the main one, that you mentioned here :)\n\nOk, no problem, we will see what happens on the mailing list.\n\nIt looks like the for-each-ref documentation has not been changed though.\n\nOtherwise it looks good to me and perhaps you could send your series\nto the mailing list even if it's long. For the first version, you may\nwant to add \"RFC\" in the subject of the patch emails you send.\n\nThanks,\n"},{"id":"337518","messageId":"CAL21Bmnir6GSgZO2TqYZY8NByNuX+8O5BEyfh5URnawhRYPGFg@mail.gmail.com","threadId":"47693","inReplyTo":"CAP8UFD31oUtoPMN+S_r5YoKrSN4p_QgZQPE6dF20Wdf6o6vdJw@mail.gmail.com","subject":"Re: Please review my code","fromName":"Оля Тележная","fromEmail":"olyatelezhnaya@gmail.com","sentAt":"2018-01-26T19:34:49Z","receivedAt":"2018-01-26T19:34:55Z","isPatch":false,"sender":{"key":"olyatelezhnaya@gmail.com","avatar":"https://avatars.githubusercontent.com/u/11246099?v=4"},"body":"2018-01-26 19:42 GMT+03:00 Christian Couder :\n> On Fri, Jan 26, 2018 at 11:32 AM, Оля Тележная wrote:\n>> 2018-01-25 23:22 GMT+03:00 Christian Couder :\n>>> On Thu, Jan 25, 2018 at 6:20 PM, Оля Тележная wrote:\n>>>> Please look at my code:\n>>>> https://github.com/telezhnaya/git/commits/catfile\n>>>> You could send me any ideas here or in Github.\n>>>\n>>> I left some comments on GitHub. My main suggestion is to try to get\n>>> rid of the is_cat global and if possible to remove the \"struct\n>>> expand_data *cat_file_info\" global.\n>>\n>> Thanks for your comments, I find them very useful. Most of issues are\n>> fixed except the main one, that you mentioned here :)\n>\n> Ok, no problem, we will see what happens on the mailing list.\n>\n> It looks like the for-each-ref documentation has not been changed though.\n\nHave you seen last commit? I updated cat-file documentation and I\nmentioned why I decided not to touch for-each-ref docs. Please share\nwith me any ideas how can I make that place better.\n\n>\n> Otherwise it looks good to me and perhaps you could send your series\n> to the mailing list even if it's long. For the first version, you may\n> want to add \"RFC\" in the subject of the patch emails you send.\n\nGreat, thanks, I will send it now.\nOlga\n\n>\n> Thanks,\n"}]}