Re: [PATCH v5 1/4] convert: Refactor would_convert_to_git() to single arg 'path'

2014-08-25 Thread Junio C Hamano
Steffen Prohaska proha...@zib.de writes:

 It is only the path that matters in the decision whether to filter or
 not.  Clarify this by making path the single argument of
 would_convert_to_git().

 Signed-off-by: Steffen Prohaska proha...@zib.de
 ---

I've retitled this as:

convert: drop arguments other than 'path' from would_convert_to_git()

to match the output from git shortlog --since=3.months --no-merges
by using lowercase 'd' after the convert:  area name, and also
more importantly avoid calling refactor which this change is not.

Thanks.

  convert.h   | 5 ++---
  sha1_file.c | 2 +-
  2 files changed, 3 insertions(+), 4 deletions(-)

 diff --git a/convert.h b/convert.h
 index 0c2143c..c638b33 100644
 --- a/convert.h
 +++ b/convert.h
 @@ -40,10 +40,9 @@ extern int convert_to_working_tree(const char *path, const 
 char *src,
  size_t len, struct strbuf *dst);
  extern int renormalize_buffer(const char *path, const char *src, size_t len,
 struct strbuf *dst);
 -static inline int would_convert_to_git(const char *path, const char *src,
 -size_t len, enum safe_crlf checksafe)
 +static inline int would_convert_to_git(const char *path)
  {
 - return convert_to_git(path, src, len, NULL, checksafe);
 + return convert_to_git(path, NULL, 0, NULL, 0);
  }
  
  /*
 diff --git a/sha1_file.c b/sha1_file.c
 index 3f70b1d..00c07f2 100644
 --- a/sha1_file.c
 +++ b/sha1_file.c
 @@ -3144,7 +3144,7 @@ int index_fd(unsigned char *sha1, int fd, struct stat 
 *st,
   if (!S_ISREG(st-st_mode))
   ret = index_pipe(sha1, fd, type, path, flags);
   else if (size = big_file_threshold || type != OBJ_BLOB ||
 -  (path  would_convert_to_git(path, NULL, 0, 0)))
 +  (path  would_convert_to_git(path)))
   ret = index_core(sha1, fd, size, type, path, flags);
   else
   ret = index_stream(sha1, fd, size, type, path, flags);
--
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 v5 1/4] convert: Refactor would_convert_to_git() to single arg 'path'

2014-08-24 Thread Steffen Prohaska
It is only the path that matters in the decision whether to filter or
not.  Clarify this by making path the single argument of
would_convert_to_git().

Signed-off-by: Steffen Prohaska proha...@zib.de
---
 convert.h   | 5 ++---
 sha1_file.c | 2 +-
 2 files changed, 3 insertions(+), 4 deletions(-)

diff --git a/convert.h b/convert.h
index 0c2143c..c638b33 100644
--- a/convert.h
+++ b/convert.h
@@ -40,10 +40,9 @@ extern int convert_to_working_tree(const char *path, const 
char *src,
   size_t len, struct strbuf *dst);
 extern int renormalize_buffer(const char *path, const char *src, size_t len,
  struct strbuf *dst);
-static inline int would_convert_to_git(const char *path, const char *src,
-  size_t len, enum safe_crlf checksafe)
+static inline int would_convert_to_git(const char *path)
 {
-   return convert_to_git(path, src, len, NULL, checksafe);
+   return convert_to_git(path, NULL, 0, NULL, 0);
 }
 
 /*
diff --git a/sha1_file.c b/sha1_file.c
index 3f70b1d..00c07f2 100644
--- a/sha1_file.c
+++ b/sha1_file.c
@@ -3144,7 +3144,7 @@ int index_fd(unsigned char *sha1, int fd, struct stat *st,
if (!S_ISREG(st-st_mode))
ret = index_pipe(sha1, fd, type, path, flags);
else if (size = big_file_threshold || type != OBJ_BLOB ||
-(path  would_convert_to_git(path, NULL, 0, 0)))
+(path  would_convert_to_git(path)))
ret = index_core(sha1, fd, size, type, path, flags);
else
ret = index_stream(sha1, fd, size, type, path, flags);
-- 
2.1.0.8.gd3b6067

--
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