* Want to start contributing to git through this task: "Use dir-iterator to avoid explicit recursiv
@ 2018-04-09 13:51 vaibhav kurhe
2018-04-09 18:05 ` Eric Sunshine
0 siblings, 1 reply; 3+ messages in thread
From: vaibhav kurhe @ 2018-04-09 13:51 UTC (permalink / raw)
To: git
Hello all,
I am a Git user since last 2 years and now I want to contribute
towards its development.
Yesterday, I had a discussion with Kevin on the IRC channel
#git-devel, where I got this link:-
https://git.github.io/SoC-2018-Microprojects/.
Out of the tasks listed down in above URL, I found this one interesting:-
"Use dir-iterator to avoid explicit recursive directory traversal"
==>
Some places in git use raw API opendir/readdir/closedir to traverse a
directory recursively, which usually involves function recursion. Now
that we have struct dir_iterator (see dir-iterator.h), convert these
to use the dir-iterator to simplify the code. Do only one conversion
per microproject.
I cloned the git repo and browsed through it. I found that in git
clone (buildtin/clone.c), raw directory API readdir() is used
recursively.
So there is a place to change it to use the already available
"dir_iterator" structure.
I just want to know, if anybody is working on this task already.
If not, then I can proceed to work.
Thank you and Regards,
Vaibhav
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: Want to start contributing to git through this task: "Use dir-iterator to avoid explicit recursiv
2018-04-09 13:51 Want to start contributing to git through this task: "Use dir-iterator to avoid explicit recursiv vaibhav kurhe
@ 2018-04-09 18:05 ` Eric Sunshine
2018-04-10 8:41 ` Johannes Schindelin
0 siblings, 1 reply; 3+ messages in thread
From: Eric Sunshine @ 2018-04-09 18:05 UTC (permalink / raw)
To: vaibhav kurhe; +Cc: Git List
On Mon, Apr 9, 2018 at 9:51 AM, vaibhav kurhe <vaibhav.kurhe@gmail.com> wrote:
> https://git.github.io/SoC-2018-Microprojects/.
> Out of the tasks listed down in above URL, I found this one interesting:-
> "Use dir-iterator to avoid explicit recursive directory traversal"
> I cloned the git repo and browsed through it. I found that in git
> clone (buildtin/clone.c), raw directory API readdir() is used
> recursively.
> So there is a place to change it to use the already available
> "dir_iterator" structure.
> I just want to know, if anybody is working on this task already.
> If not, then I can proceed to work.
There has been no recent discussion on the mailing list regarding
this, so it is unlikely that anyone else is working on it.
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: Want to start contributing to git through this task: "Use dir-iterator to avoid explicit recursiv
2018-04-09 18:05 ` Eric Sunshine
@ 2018-04-10 8:41 ` Johannes Schindelin
0 siblings, 0 replies; 3+ messages in thread
From: Johannes Schindelin @ 2018-04-10 8:41 UTC (permalink / raw)
To: Eric Sunshine; +Cc: vaibhav kurhe, Git List
Hi,
On Mon, 9 Apr 2018, Eric Sunshine wrote:
> On Mon, Apr 9, 2018 at 9:51 AM, vaibhav kurhe <vaibhav.kurhe@gmail.com> wrote:
> > https://git.github.io/SoC-2018-Microprojects/.
> > Out of the tasks listed down in above URL, I found this one interesting:-
> > "Use dir-iterator to avoid explicit recursive directory traversal"
> > I cloned the git repo and browsed through it. I found that in git
> > clone (buildtin/clone.c), raw directory API readdir() is used
> > recursively.
> > So there is a place to change it to use the already available
> > "dir_iterator" structure.
> > I just want to know, if anybody is working on this task already.
> > If not, then I can proceed to work.
>
> There has been no recent discussion on the mailing list regarding
> this, so it is unlikely that anyone else is working on it.
Indeed.
Please feel free to reach out when you get stuck. If you want to show the
code in progress, may I suggest to publish your topic branch to a public
repository, such as on GitHub?
Ciao,
Johannes
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2018-04-10 8:41 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-04-09 13:51 Want to start contributing to git through this task: "Use dir-iterator to avoid explicit recursiv vaibhav kurhe
2018-04-09 18:05 ` Eric Sunshine
2018-04-10 8:41 ` Johannes Schindelin
Code repositories for project(s) associated with this public inbox
https://80x24.org/mirrors/git.git
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).