{"thread":{"id":"48251","subject":"Want to start contributing to git through this task: \"Use dir-iterator to avoid explicit recursiv","startedAt":"2018-04-09T13:52:09Z","lastAt":"2018-04-10T08:41:39Z","messageCount":3,"participants":["vaibhav kurhe","Eric Sunshine","Johannes Schindelin"],"isPatch":false,"patchVersion":null,"patchTotal":null},"messages":[{"id":"344195","messageId":"CAHKv1qs-cnmcd2pR9iv=Xx1aKnb=SEsCQ7OAjy-4yHhCE_e_dw@mail.gmail.com","threadId":"48251","inReplyTo":null,"subject":"Want to start contributing to git through this task: \"Use dir-iterator to avoid explicit recursiv","fromName":"vaibhav kurhe","fromEmail":"vaibhav.kurhe@gmail.com","sentAt":"2018-04-09T13:51:44Z","receivedAt":"2018-04-09T13:52:09Z","isPatch":false,"sender":{"key":"vaibhav.kurhe@gmail.com","avatar":null},"body":"Hello all,\nI am a Git user since last 2 years and now I want to contribute\ntowards its development.\nYesterday, I had a discussion with Kevin on the IRC channel\n#git-devel, where I got this link:-\nhttps://git.github.io/SoC-2018-Microprojects/.\nOut of the tasks listed down in above URL, I found this one interesting:-\n\"Use dir-iterator to avoid explicit recursive directory traversal\"\n==>\nSome places in git use raw API opendir/readdir/closedir to traverse a\ndirectory recursively, which usually involves function recursion. Now\nthat we have struct dir_iterator (see dir-iterator.h), convert these\nto use the dir-iterator to simplify the code. Do only one conversion\nper microproject.\n\nI cloned the git repo and browsed through it. I found that in git\nclone (buildtin/clone.c), raw directory API readdir() is used\nrecursively.\nSo there is a place to change it to use the already available\n\"dir_iterator\" structure.\n\nI just want to know, if anybody is working on this task already.\nIf not, then I can proceed to work.\n\nThank you and Regards,\nVaibhav\n"},{"id":"344221","messageId":"CAPig+cRNbDBpDn_ku0cXJhzvnrSOkBTztp8tWomePxfXaviotw@mail.gmail.com","threadId":"48251","inReplyTo":"CAHKv1qs-cnmcd2pR9iv=Xx1aKnb=SEsCQ7OAjy-4yHhCE_e_dw@mail.gmail.com","subject":"Re: Want to start contributing to git through this task: \"Use dir-iterator to avoid explicit recursiv","fromName":"Eric Sunshine","fromEmail":"sunshine@sunshineco.com","sentAt":"2018-04-09T18:05:47Z","receivedAt":"2018-04-09T18:05:53Z","isPatch":false,"sender":{"key":"sunshine@sunshineco.com","avatar":"https://avatars.githubusercontent.com/u/163641?v=4"},"body":"On Mon, Apr 9, 2018 at 9:51 AM, vaibhav kurhe <vaibhav.kurhe@gmail.com> wrote:\n> https://git.github.io/SoC-2018-Microprojects/.\n> Out of the tasks listed down in above URL, I found this one interesting:-\n> \"Use dir-iterator to avoid explicit recursive directory traversal\"\n> I cloned the git repo and browsed through it. I found that in git\n> clone (buildtin/clone.c), raw directory API readdir() is used\n> recursively.\n> So there is a place to change it to use the already available\n> \"dir_iterator\" structure.\n> I just want to know, if anybody is working on this task already.\n> If not, then I can proceed to work.\n\nThere has been no recent discussion on the mailing list regarding\nthis, so it is unlikely that anyone else is working on it.\n"},{"id":"344305","messageId":"nycvar.QRO.7.76.6.1804101040090.55@ZVAVAG-6OXH6DA.rhebcr.pbec.zvpebfbsg.pbz","threadId":"48251","inReplyTo":"CAPig+cRNbDBpDn_ku0cXJhzvnrSOkBTztp8tWomePxfXaviotw@mail.gmail.com","subject":"Re: Want to start contributing to git through this task: \"Use dir-iterator to avoid explicit recursiv","fromName":"Johannes Schindelin","fromEmail":"johannes.schindelin@gmx.de","sentAt":"2018-04-10T08:41:30Z","receivedAt":"2018-04-10T08:41:39Z","isPatch":false,"sender":{"key":"johannes.schindelin@gmx.de","avatar":"https://avatars.githubusercontent.com/u/127790?v=4"},"body":"Hi,\n\nOn Mon, 9 Apr 2018, Eric Sunshine wrote:\n\n> On Mon, Apr 9, 2018 at 9:51 AM, vaibhav kurhe <vaibhav.kurhe@gmail.com> wrote:\n> > https://git.github.io/SoC-2018-Microprojects/.\n> > Out of the tasks listed down in above URL, I found this one interesting:-\n> > \"Use dir-iterator to avoid explicit recursive directory traversal\"\n> > I cloned the git repo and browsed through it. I found that in git\n> > clone (buildtin/clone.c), raw directory API readdir() is used\n> > recursively.\n> > So there is a place to change it to use the already available\n> > \"dir_iterator\" structure.\n> > I just want to know, if anybody is working on this task already.\n> > If not, then I can proceed to work.\n> \n> There has been no recent discussion on the mailing list regarding\n> this, so it is unlikely that anyone else is working on it.\n\nIndeed.\n\nPlease feel free to reach out when you get stuck. If you want to show the\ncode in progress, may I suggest to publish your topic branch to a public\nrepository, such as on GitHub?\n\nCiao,\nJohannes\n"}]}