Re: [PATCH] microproject for GSOC
沈承恩 writes: > Thank you for your comments.I will amend those issues .As a Chinese > student,what name should I use?My Chinese name is ok? I guess it is. The git.git history already has contribution from 张忠山 for example (I don't know if it is chineese, but it looks so from my French eyes). > I am very interest in Open source.What can I do to increase my chance? Interesting blog post to read: http://softwareswirl.blogspot.fr/2014/03/my-secret-tip-for-gsoc-success.html -- Matthieu Moy http://www-verimag.imag.fr/~moy/ -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re:Re: [PATCH] microproject for GSOC
Thank you for your comments.I will amend those issues .As a Chinese student,what name should I use?My Chinese name is ok? At 2014-03-15 01:10:57,"Matthieu Moy" wrote: >Hi, > >Welcome to the Git community, and welcome to the GSOC program. Below are >some comments to give you a taste of what a review looks like on this >list. Do take the comments seriously (they should be addressed), but >don't take them badly: critic is meant to be constructive. > >ubuntu733 writes: >^ > >Please, use a real name when you contribute to Git. > >> Apply for GSOC.The microprojects is rewriter diff-index.c > >This part of your message will become the commit message (i.e. cast in >stone forever in git.git's history). The point is not that you want to >apply for GSOC, but what the patch does and more importantly why it does >it. > >> +#define REMOVE 1 > >If the code is to be removed, then remove it. That's why we use a >version control system ;-). > >> -while ((e = readdir(dir))) >> -if (strcmp(".", e->d_name) && strcmp("..", e->d_name)) >> -string_list_insert(list, e->d_name); >> - >> +while ((e = readdir(dir))) { >> +while(is_dot_or_dotdot(e->d_name)) > >Missing space between "while" and "(". > >> + break; > >Broken indentation (indent with space). > >This while (...) break; seems really weird to me: if the condition is >false, then you exit the loop because it's a while loop, and if the >condition is true, you exit the loop because of the break. Isn't that a >no-op? > >> +string_list_insert(list, e->d_name); >> + } > >Broken indentation (misplaced }). > >-- >Matthieu Moy >http://www-verimag.imag.fr/~moy/
Re:Re: [PATCH] microproject for GSOC
Thank you for your comments.I will amend those issues .As a Chinese student,what name should I use?My Chinese name is ok? I am very interest in Open source.What can I do to increase my chance? At 2014-03-15 01:10:57,"Matthieu Moy" wrote: >Hi, > >Welcome to the Git community, and welcome to the GSOC program. Below are >some comments to give you a taste of what a review looks like on this >list. Do take the comments seriously (they should be addressed), but >don't take them badly: critic is meant to be constructive. > >ubuntu733 writes: >^ > >Please, use a real name when you contribute to Git. > >> Apply for GSOC.The microprojects is rewriter diff-index.c > >This part of your message will become the commit message (i.e. cast in >stone forever in git.git's history). The point is not that you want to >apply for GSOC, but what the patch does and more importantly why it does >it. > >> +#define REMOVE 1 > >If the code is to be removed, then remove it. That's why we use a >version control system ;-). > >> -while ((e = readdir(dir))) >> -if (strcmp(".", e->d_name) && strcmp("..", e->d_name)) >> -string_list_insert(list, e->d_name); >> - >> +while ((e = readdir(dir))) { >> +while(is_dot_or_dotdot(e->d_name)) > >Missing space between "while" and "(". > >> + break; > >Broken indentation (indent with space). > >This while (...) break; seems really weird to me: if the condition is >false, then you exit the loop because it's a while loop, and if the >condition is true, you exit the loop because of the break. Isn't that a >no-op? > >> +string_list_insert(list, e->d_name); >> + } > >Broken indentation (misplaced }). > >-- >Matthieu Moy >http://www-verimag.imag.fr/~moy/
Re: [PATCH] microproject for GSOC
Hi, Welcome to the Git community, and welcome to the GSOC program. Below are some comments to give you a taste of what a review looks like on this list. Do take the comments seriously (they should be addressed), but don't take them badly: critic is meant to be constructive. ubuntu733 writes: ^ Please, use a real name when you contribute to Git. > Apply for GSOC.The microprojects is rewriter diff-index.c This part of your message will become the commit message (i.e. cast in stone forever in git.git's history). The point is not that you want to apply for GSOC, but what the patch does and more importantly why it does it. > +#define REMOVE 1 If the code is to be removed, then remove it. That's why we use a version control system ;-). > - while ((e = readdir(dir))) > - if (strcmp(".", e->d_name) && strcmp("..", e->d_name)) > - string_list_insert(list, e->d_name); > - > + while ((e = readdir(dir))) { > + while(is_dot_or_dotdot(e->d_name)) Missing space between "while" and "(". > + break; Broken indentation (indent with space). This while (...) break; seems really weird to me: if the condition is false, then you exit the loop because it's a while loop, and if the condition is true, you exit the loop because of the break. Isn't that a no-op? > + string_list_insert(list, e->d_name); > + } Broken indentation (misplaced }). -- Matthieu Moy http://www-verimag.imag.fr/~moy/ -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
[PATCH] microproject for GSOC
Apply for GSOC.The microprojects is rewriter diff-index.c Signed-off-by: ubuntu733 --- diff-no-index.c | 11 ++- dir.h |3 ++- 2 files changed, 8 insertions(+), 6 deletions(-) diff --git a/diff-no-index.c b/diff-no-index.c index 8e10bff..91ece64 100644 --- a/diff-no-index.c +++ b/diff-no-index.c @@ -3,7 +3,7 @@ * Copyright (c) 2007 by Johannes Schindelin * Copyright (c) 2008 by Junio C Hamano */ - +#define REMOVE 1 #include "cache.h" #include "color.h" #include "commit.h" @@ -24,10 +24,11 @@ static int read_directory(const char *path, struct string_list *list) if (!(dir = opendir(path))) return error("Could not open directory %s", path); - while ((e = readdir(dir))) - if (strcmp(".", e->d_name) && strcmp("..", e->d_name)) - string_list_insert(list, e->d_name); - + while ((e = readdir(dir))) { + while(is_dot_or_dotdot(e->d_name)) + break; + string_list_insert(list, e->d_name); + } closedir(dir); return 0; } diff --git a/dir.h b/dir.h index 55e5345..1d68818 100644 --- a/dir.h +++ b/dir.h @@ -138,8 +138,9 @@ extern int match_pathspec(const struct pathspec *pathspec, extern int within_depth(const char *name, int namelen, int depth, int max_depth); extern int fill_directory(struct dir_struct *dir, const struct pathspec *pathspec); +#ifndef REMOVE extern int read_directory(struct dir_struct *, const char *path, int len, const struct pathspec *pathspec); - +#endif extern int is_excluded_from_list(const char *pathname, int pathlen, const char *basename, int *dtype, struct exclude_list *el); struct dir_entry *dir_add_ignored(struct dir_struct *dir, const char *pathname, int len); -- 1.7.9.5 -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html