git@vger.kernel.org mailing list mirror (one of many)
 help / color / mirror / code / Atom feed
From: Denton Liu <liu.denton@gmail.com>
To: Git Mailing List <git@vger.kernel.org>
Cc: Junio C Hamano <gitster@pobox.com>, Jeff King <peff@peff.net>,
	Emily Shaffer <emilyshaffer@google.com>,
	Thomas Gummerer <t.gummerer@gmail.com>
Subject: [PATCH v4 0/4] remove extern from function declarations
Date: Tue, 23 Apr 2019 16:40:41 -0700	[thread overview]
Message-ID: <cover.1556062365.git.liu.denton@gmail.com> (raw)
In-Reply-To: <cover.1555352526.git.liu.denton@gmail.com>

Thanks for the review, Peff.

I've excluded all the files under compat/ from being changed as it's
easier this way. After this patchset is merged, I'll probably get around
to removing the externs from compat/ files later (the ones that aren't
copied from some external source) but for now, this patchset is big
enough as it is so we can leave those for now.

I also opted not to remove the noextern.cocci since we can probably do
this in the future but it's probably beneficial to leave it in for now
for the reasons that you've mentioned.

---

Changes since v1:

* Used spatch with sed instead of sed alone
* Fixed sed expression to ignore function variables

Changes since v2:

* Rebased on latest master (since last patchset hasn't been picked up
  yet)
* Manually aligned parameter lists that were mangled by the tools

Changes since v3:

* Excluded compat/ from cleanup

Denton Liu (4):
  *.[ch]: remove extern from function declarations using spatch
  *.[ch]: remove extern from function declarations using sed
  *.[ch]: manually align parameter lists
  cocci: prevent extern function declarations

 advice.h                          |   2 +-
 archive.h                         |  24 +-
 bisect.h                          |  26 +-
 blame.h                           |   2 +-
 branch.h                          |  14 +-
 builtin.h                         | 254 ++++++++++----------
 bulk-checkin.h                    |  10 +-
 cache.h                           | 386 +++++++++++++++---------------
 checkout.h                        |   6 +-
 column.h                          |  16 +-
 commit.h                          | 116 ++++-----
 config.h                          | 226 ++++++++---------
 connect.h                         |  22 +-
 contrib/coccinelle/noextern.cocci |   6 +
 csum-file.h                       |  20 +-
 decorate.h                        |   4 +-
 delta.h                           |  14 +-
 dir.h                             | 144 +++++------
 exec-cmd.h                        |  16 +-
 fmt-merge-msg.h                   |   2 +-
 fsmonitor.h                       |  14 +-
 gettext.h                         |   8 +-
 git-compat-util.h                 | 132 +++++-----
 grep.h                            |  22 +-
 hashmap.h                         |  30 +--
 help.h                            |  36 +--
 http.h                            |  62 ++---
 khash.h                           |  16 +-
 kwset.h                           |  10 +-
 line-log.h                        |  16 +-
 lockfile.h                        |  12 +-
 ls-refs.h                         |   4 +-
 mailinfo.h                        |   6 +-
 merge-blobs.h                     |   6 +-
 object-store.h                    |  32 +--
 object.h                          |  12 +-
 oidmap.h                          |  12 +-
 pack.h                            |  26 +-
 packfile.h                        |  82 +++----
 path.h                            |  42 ++--
 pkt-line.h                        |  10 +-
 ppc/sha1.c                        |   4 +-
 prio-queue.h                      |  10 +-
 protocol.h                        |   6 +-
 quote.h                           |  34 +--
 reachable.h                       |   8 +-
 reflog-walk.h                     |  28 +--
 refs.h                            |   2 +-
 remote.h                          |  24 +-
 replace-object.h                  |   4 +-
 resolve-undo.h                    |  14 +-
 run-command.h                     |   8 +-
 serve.h                           |   6 +-
 sha1-lookup.h                     |   8 +-
 streaming.h                       |   8 +-
 string-list.h                     |   4 +-
 sub-process.h                     |   8 +-
 submodule-config.h                |  22 +-
 tag.h                             |  16 +-
 tempfile.h                        |  30 +--
 trace.h                           |  44 ++--
 transport.h                       |   4 +-
 tree-walk.h                       |   4 +-
 upload-pack.h                     |   8 +-
 url.h                             |  16 +-
 urlmatch.h                        |   4 +-
 utf8.h                            |   2 +-
 varint.h                          |   4 +-
 vcs-svn/sliding_window.h          |   2 +-
 vcs-svn/svndiff.h                 |   4 +-
 worktree.h                        |  36 +--
 xdiff-interface.h                 |  12 +-
 72 files changed, 1145 insertions(+), 1139 deletions(-)
 create mode 100644 contrib/coccinelle/noextern.cocci

Range-diff against v3:
1:  2ed74c5e5b ! 1:  720d50aea7 *.[ch]: remove extern from function declarations using spatch
    @@ -19,7 +19,13 @@
     
         and it was run with:
     
    -            $ spatch --sp-file contrib/coccinelle/noextern.cocci --dir . --include-headers --in-place
    +            $ git ls-files \*.{c,h} | \
    +                    grep -v ^compat/ | \
    +                    xargs spatch --sp-file contrib/coccinelle/noextern.cocci --in-place
    +
    +    Files under `compat/` are intentionally excluded as some are directly
    +    copied from external sources and we should avoid churning them as much
    +    as possible.
     
         Signed-off-by: Denton Liu <liu.denton@gmail.com>
     
    @@ -1388,242 +1394,6 @@
      /* record author-date for each commit object */
      struct author_date_slab;
     
    - diff --git a/compat/mingw.c b/compat/mingw.c
    - --- a/compat/mingw.c
    - +++ b/compat/mingw.c
    -@@
    - 	int newmode;
    - } _startupinfo;
    - 
    --extern int __wgetmainargs(int *argc, wchar_t ***argv, wchar_t ***env, int glob,
    -+int __wgetmainargs(int *argc, wchar_t ***argv, wchar_t ***env, int glob,
    - 		_startupinfo *si);
    - 
    - static NORETURN void die_startup(void)
    -
    - diff --git a/compat/mingw.h b/compat/mingw.h
    - --- a/compat/mingw.h
    - +++ b/compat/mingw.h
    -@@
    - #undef _POSIX_THREAD_SAFE_FUNCTIONS
    - #endif
    - 
    --extern int mingw_core_config(const char *var, const char *value, void *cb);
    -+int mingw_core_config(const char *var, const char *value, void *cb);
    - #define platform_core_config mingw_core_config
    - 
    - /*
    -@@
    - 			*path = '/';
    - }
    - #define PATH_SEP ';'
    --extern char *mingw_query_user_email(void);
    -+char *mingw_query_user_email(void);
    - #define query_user_email mingw_query_user_email
    - #if !defined(__MINGW64_VERSION_MAJOR) && (!defined(_MSC_VER) || _MSC_VER < 1800)
    - #define PRIuMAX "I64u"
    -@@
    - /*
    -  * Used by Pthread API implementation for Windows
    -  */
    --extern int err_win_to_posix(DWORD winerr);
    -+int err_win_to_posix(DWORD winerr);
    -
    - diff --git a/compat/nedmalloc/malloc.c.h b/compat/nedmalloc/malloc.c.h
    - --- a/compat/nedmalloc/malloc.c.h
    - +++ b/compat/nedmalloc/malloc.c.h
    -@@
    - #include <unistd.h>     /* for sbrk, sysconf */
    - #else /* LACKS_UNISTD_H */
    - #if !defined(__FreeBSD__) && !defined(__OpenBSD__) && !defined(__NetBSD__)
    --extern void*     sbrk(ptrdiff_t);
    -+void*     sbrk(ptrdiff_t);
    - #endif /* FreeBSD etc */
    - #endif /* LACKS_UNISTD_H */
    - 
    -@@
    - #    define malloc_getpagesize sysconf(_SC_PAGE_SIZE)
    - #  else
    - #    if defined(BSD) || defined(DGUX) || defined(HAVE_GETPAGESIZE)
    --       extern size_t getpagesize();
    -+       size_t getpagesize();
    - #      define malloc_getpagesize getpagesize()
    - #    else
    - #      ifdef WIN32 /* use supplied emulation of getpagesize */
    -@@
    - /* skipped internal declaration from pthread.h */
    - #ifdef linux
    - #ifndef PTHREAD_MUTEX_RECURSIVE
    --extern int pthread_mutexattr_setkind_np __P ((pthread_mutexattr_t *__attr,
    -+int pthread_mutexattr_setkind_np __P ((pthread_mutexattr_t *__attr,
    - 					   int __kind));
    - #define PTHREAD_MUTEX_RECURSIVE PTHREAD_MUTEX_RECURSIVE_NP
    - #define pthread_mutexattr_settype(x,y) pthread_mutexattr_setkind_np(x,y)
    -
    - diff --git a/compat/obstack.h b/compat/obstack.h
    - --- a/compat/obstack.h
    - +++ b/compat/obstack.h
    -@@
    - 
    - /* Declare the external functions we use; they are in obstack.c.  */
    - 
    --extern void _obstack_newchunk (struct obstack *, int);
    --extern int _obstack_begin (struct obstack *, int, int,
    -+void _obstack_newchunk (struct obstack *, int);
    -+int _obstack_begin (struct obstack *, int, int,
    - 			    void *(*) (long), void (*) (void *));
    --extern int _obstack_begin_1 (struct obstack *, int, int,
    -+int _obstack_begin_1 (struct obstack *, int, int,
    - 			     void *(*) (void *, long),
    - 			     void (*) (void *, void *), void *);
    --extern int _obstack_memory_used (struct obstack *);
    -+int _obstack_memory_used (struct obstack *);
    - 
    - void obstack_free (struct obstack *, void *);
    - 
    -
    - diff --git a/compat/poll/poll.h b/compat/poll/poll.h
    - --- a/compat/poll/poll.h
    - +++ b/compat/poll/poll.h
    -@@
    - 
    - typedef unsigned long nfds_t;
    - 
    --extern int poll (struct pollfd *pfd, nfds_t nfd, int timeout);
    -+int poll (struct pollfd *pfd, nfds_t nfd, int timeout);
    - 
    - /* Define INFTIM only if doing so conforms to POSIX.  */
    - #if !defined (_POSIX_C_SOURCE) && !defined (_XOPEN_SOURCE)
    -
    - diff --git a/compat/regex/regex.h b/compat/regex/regex.h
    - --- a/compat/regex/regex.h
    - +++ b/compat/regex/regex.h
    -@@
    - #ifdef __USE_GNU
    - /* Sets the current default syntax to SYNTAX, and return the old syntax.
    -    You can also simply assign to the `re_syntax_options' variable.  */
    --extern reg_syntax_t re_set_syntax (reg_syntax_t __syntax);
    -+reg_syntax_t re_set_syntax (reg_syntax_t __syntax);
    - 
    - /* Compile the regular expression PATTERN, with length LENGTH
    -    and syntax given by the global `re_syntax_options', into the buffer
    -    BUFFER.  Return NULL if successful, and an error string if not.  */
    --extern const char *re_compile_pattern (const char *__pattern, size_t __length,
    -+const char *re_compile_pattern (const char *__pattern, size_t __length,
    - 				       struct re_pattern_buffer *__buffer);
    - 
    - 
    - /* Compile a fastmap for the compiled pattern in BUFFER; used to
    -    accelerate searches.  Return 0 if successful and -2 if was an
    -    internal error.  */
    --extern int re_compile_fastmap (struct re_pattern_buffer *__buffer);
    -+int re_compile_fastmap (struct re_pattern_buffer *__buffer);
    - 
    - 
    - /* Search in the string STRING (with length LENGTH) for the pattern
    -@@
    -    characters.  Return the starting position of the match, -1 for no
    -    match, or -2 for an internal error.  Also return register
    -    information in REGS (if REGS and BUFFER->no_sub are nonzero).  */
    --extern int re_search (struct re_pattern_buffer *__buffer, const char *__cstring,
    -+int re_search (struct re_pattern_buffer *__buffer, const char *__cstring,
    - 		      int __length, int __start, int __range,
    - 		      struct re_registers *__regs);
    - 
    - 
    - /* Like `re_search', but search in the concatenation of STRING1 and
    -    STRING2.  Also, stop searching at index START + STOP.  */
    --extern int re_search_2 (struct re_pattern_buffer *__buffer,
    -+int re_search_2 (struct re_pattern_buffer *__buffer,
    - 			const char *__string1, int __length1,
    - 			const char *__string2, int __length2, int __start,
    - 			int __range, struct re_registers *__regs, int __stop);
    -@@
    - 
    - /* Like `re_search', but return how many characters in STRING the regexp
    -    in BUFFER matched, starting at position START.  */
    --extern int re_match (struct re_pattern_buffer *__buffer, const char *__cstring,
    -+int re_match (struct re_pattern_buffer *__buffer, const char *__cstring,
    - 		     int __length, int __start, struct re_registers *__regs);
    - 
    - 
    - /* Relates to `re_match' as `re_search_2' relates to `re_search'.  */
    --extern int re_match_2 (struct re_pattern_buffer *__buffer,
    -+int re_match_2 (struct re_pattern_buffer *__buffer,
    - 		       const char *__string1, int __length1,
    - 		       const char *__string2, int __length2, int __start,
    - 		       struct re_registers *__regs, int __stop);
    -@@
    -    Unless this function is called, the first search or match using
    -    PATTERN_BUFFER will allocate its own register data, without
    -    freeing the old data.  */
    --extern void re_set_registers (struct re_pattern_buffer *__buffer,
    -+void re_set_registers (struct re_pattern_buffer *__buffer,
    - 			      struct re_registers *__regs,
    - 			      unsigned int __num_regs,
    - 			      regoff_t *__starts, regoff_t *__ends);
    -@@
    - #if defined _REGEX_RE_COMP || (defined _LIBC && defined __USE_BSD)
    - # ifndef _CRAY
    - /* 4.2 bsd compatibility.  */
    --extern char *re_comp (const char *);
    --extern int re_exec (const char *);
    -+char *re_comp (const char *);
    -+int re_exec (const char *);
    - # endif
    - #endif
    - 
    -@@
    - #endif
    - 
    - /* POSIX compatibility.  */
    --extern int regcomp (regex_t *__restrict __preg,
    -+int regcomp (regex_t *__restrict __preg,
    - 		    const char *__restrict __pattern,
    - 		    int __cflags);
    - 
    --extern int regexec (const regex_t *__restrict __preg,
    -+int regexec (const regex_t *__restrict __preg,
    - 		    const char *__restrict __cstring, size_t __nmatch,
    - 		    regmatch_t __pmatch[__restrict_arr],
    - 		    int __eflags);
    - 
    --extern size_t regerror (int __errcode, const regex_t *__restrict __preg,
    -+size_t regerror (int __errcode, const regex_t *__restrict __preg,
    - 			char *__restrict __errbuf, size_t __errbuf_size);
    - 
    --extern void regfree (regex_t *__preg);
    -+void regfree (regex_t *__preg);
    - 
    - 
    - #ifdef __cplusplus
    -
    - diff --git a/compat/win32/pthread.h b/compat/win32/pthread.h
    - --- a/compat/win32/pthread.h
    - +++ b/compat/win32/pthread.h
    -@@
    - 	DWORD tid;
    - } pthread_t;
    - 
    --extern int pthread_create(pthread_t *thread, const void *unused,
    -+int pthread_create(pthread_t *thread, const void *unused,
    - 			  void *(*start_routine)(void*), void *arg);
    - 
    - /*
    -@@
    -  */
    - #define pthread_join(a, b) win32_pthread_join(&(a), (b))
    - 
    --extern int win32_pthread_join(pthread_t *thread, void **value_ptr);
    -+int win32_pthread_join(pthread_t *thread, void **value_ptr);
    - 
    - #define pthread_equal(t1, t2) ((t1).tid == (t2).tid)
    --extern pthread_t pthread_self(void);
    -+pthread_t pthread_self(void);
    - 
    - static inline void NORETURN pthread_exit(void *ret)
    - {
    -
      diff --git a/config.h b/config.h
      --- a/config.h
      +++ b/config.h
2:  7d35a052a6 ! 2:  0ebda47252 *.[ch]: remove extern from function declarations using sed
    @@ -8,7 +8,13 @@
     
         This was done by running the following on my system with sed 4.2.2:
     
    -        $ git ls-files \*.{c,h} | xargs sed -i'' -e 's/^\(\s*\)extern \([^(]*([^*]\)/\1\2/'
    +        $ git ls-files \*.{c,h} | \
    +            grep -v ^compat/ | \
    +            xargs sed -i'' -e 's/^\(\s*\)extern \([^(]*([^*]\)/\1\2/'
    +
    +    Files under `compat/` are intentionally excluded as some are directly
    +    copied from external sources and we should avoid churning them as much
    +    as possible.
     
         Then, leftover instances of extern were found by running
     
3:  916c34450f ! 3:  a813c36cab *.[ch]: manually align parameter lists
    @@ -391,125 +391,6 @@
      
      /*
     
    - diff --git a/compat/obstack.h b/compat/obstack.h
    - --- a/compat/obstack.h
    - +++ b/compat/obstack.h
    -@@
    - 
    - void _obstack_newchunk (struct obstack *, int);
    - int _obstack_begin (struct obstack *, int, int,
    --			    void *(*) (long), void (*) (void *));
    -+		    void *(*) (long), void (*) (void *));
    - int _obstack_begin_1 (struct obstack *, int, int,
    --			     void *(*) (void *, long),
    --			     void (*) (void *, void *), void *);
    -+		      void *(*) (void *, long),
    -+		      void (*) (void *, void *), void *);
    - int _obstack_memory_used (struct obstack *);
    - 
    - void obstack_free (struct obstack *, void *);
    -
    - diff --git a/compat/regex/regex.h b/compat/regex/regex.h
    - --- a/compat/regex/regex.h
    - +++ b/compat/regex/regex.h
    -@@
    -    and syntax given by the global `re_syntax_options', into the buffer
    -    BUFFER.  Return NULL if successful, and an error string if not.  */
    - const char *re_compile_pattern (const char *__pattern, size_t __length,
    --				       struct re_pattern_buffer *__buffer);
    -+				struct re_pattern_buffer *__buffer);
    - 
    - 
    - /* Compile a fastmap for the compiled pattern in BUFFER; used to
    -@@
    -    match, or -2 for an internal error.  Also return register
    -    information in REGS (if REGS and BUFFER->no_sub are nonzero).  */
    - int re_search (struct re_pattern_buffer *__buffer, const char *__cstring,
    --		      int __length, int __start, int __range,
    --		      struct re_registers *__regs);
    -+	       int __length, int __start, int __range,
    -+	       struct re_registers *__regs);
    - 
    - 
    - /* Like `re_search', but search in the concatenation of STRING1 and
    -    STRING2.  Also, stop searching at index START + STOP.  */
    - int re_search_2 (struct re_pattern_buffer *__buffer,
    --			const char *__string1, int __length1,
    --			const char *__string2, int __length2, int __start,
    --			int __range, struct re_registers *__regs, int __stop);
    -+		 const char *__string1, int __length1,
    -+		 const char *__string2, int __length2, int __start,
    -+		 int __range, struct re_registers *__regs, int __stop);
    - 
    - 
    - /* Like `re_search', but return how many characters in STRING the regexp
    -    in BUFFER matched, starting at position START.  */
    - int re_match (struct re_pattern_buffer *__buffer, const char *__cstring,
    --		     int __length, int __start, struct re_registers *__regs);
    -+	      int __length, int __start, struct re_registers *__regs);
    - 
    - 
    - /* Relates to `re_match' as `re_search_2' relates to `re_search'.  */
    - int re_match_2 (struct re_pattern_buffer *__buffer,
    --		       const char *__string1, int __length1,
    --		       const char *__string2, int __length2, int __start,
    --		       struct re_registers *__regs, int __stop);
    -+		const char *__string1, int __length1,
    -+		const char *__string2, int __length2, int __start,
    -+		struct re_registers *__regs, int __stop);
    - 
    - 
    - /* Set REGS to hold NUM_REGS registers, storing them in STARTS and
    -@@
    -    PATTERN_BUFFER will allocate its own register data, without
    -    freeing the old data.  */
    - void re_set_registers (struct re_pattern_buffer *__buffer,
    --			      struct re_registers *__regs,
    --			      unsigned int __num_regs,
    --			      regoff_t *__starts, regoff_t *__ends);
    -+		       struct re_registers *__regs,
    -+		       unsigned int __num_regs,
    -+		       regoff_t *__starts, regoff_t *__ends);
    - #endif	/* Use GNU */
    - 
    - #if defined _REGEX_RE_COMP || (defined _LIBC && defined __USE_BSD)
    -@@
    - 
    - /* POSIX compatibility.  */
    - int regcomp (regex_t *__restrict __preg,
    --		    const char *__restrict __pattern,
    --		    int __cflags);
    -+	     const char *__restrict __pattern,
    -+	     int __cflags);
    - 
    - int regexec (const regex_t *__restrict __preg,
    --		    const char *__restrict __cstring, size_t __nmatch,
    --		    regmatch_t __pmatch[__restrict_arr],
    --		    int __eflags);
    -+	     const char *__restrict __cstring, size_t __nmatch,
    -+	     regmatch_t __pmatch[__restrict_arr],
    -+	     int __eflags);
    - 
    - size_t regerror (int __errcode, const regex_t *__restrict __preg,
    --			char *__restrict __errbuf, size_t __errbuf_size);
    -+		 char *__restrict __errbuf, size_t __errbuf_size);
    - 
    - void regfree (regex_t *__preg);
    - 
    -
    - diff --git a/compat/win32/pthread.h b/compat/win32/pthread.h
    - --- a/compat/win32/pthread.h
    - +++ b/compat/win32/pthread.h
    -@@
    - } pthread_t;
    - 
    - int pthread_create(pthread_t *thread, const void *unused,
    --			  void *(*start_routine)(void*), void *arg);
    -+		   void *(*start_routine)(void*), void *arg);
    - 
    - /*
    -  * To avoid the need of copying a struct, we use small macro wrapper to pass
    -
      diff --git a/config.h b/config.h
      --- a/config.h
      +++ b/config.h
4:  d5a1663048 = 4:  23ed6ad679 cocci: prevent extern function declarations
-- 
2.21.0.1000.g7817e26e80


  parent reply	other threads:[~2019-04-23 23:40 UTC|newest]

Thread overview: 103+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-04-12 23:48 [RFC PATCH] *.h: remove extern from function declarations Denton Liu
2019-04-13  1:24 ` Jeff King
2019-04-13  5:45   ` Junio C Hamano
2019-04-15 18:24 ` [PATCH v2 0/3] " Denton Liu
2019-04-15 18:24   ` [PATCH v2 1/3] *.[ch]: remove extern from function declarations using spatch Denton Liu
2019-04-15 19:19     ` Thomas Gummerer
2019-04-15 18:24   ` [PATCH v2 2/3] *.[ch]: remove extern from function declarations using sed Denton Liu
2019-04-15 18:24   ` [PATCH v2 3/3] cocci: prevent extern function declarations Denton Liu
2019-04-17  7:58   ` [PATCH v3 0/4] remove extern from " Denton Liu
2019-04-17  7:58     ` [PATCH v3 1/4] *.[ch]: remove extern from function declarations using spatch Denton Liu
2019-04-17  7:58     ` [PATCH v3 2/4] *.[ch]: remove extern from function declarations using sed Denton Liu
2019-04-17  7:58     ` [PATCH v3 3/4] *.[ch]: manually align parameter lists Denton Liu
2019-04-17  7:58     ` [PATCH v3 4/4] cocci: prevent extern function declarations Denton Liu
2019-04-22  5:44     ` [PATCH] cache.h: fix mismerge of 'dl/no-extern-in-func-decl' Denton Liu
2019-04-22  6:30       ` Junio C Hamano
2019-04-22 11:19         ` Junio C Hamano
2019-04-22 21:49     ` [PATCH v3 0/4] remove extern from function declarations Jeff King
2019-04-25 12:07       ` SZEDER Gábor
2019-04-25 18:05         ` Denton Liu
2019-04-30 23:21         ` Johannes Schindelin
2019-05-01 10:01           ` Denton Liu
2019-05-01 18:56             ` Jeff King
2019-05-02  0:04             ` SZEDER Gábor
2019-05-03  9:32               ` Johannes Schindelin
2019-05-03 14:42                 ` SZEDER Gábor
2019-05-03 14:58                   ` SZEDER Gábor
2019-05-03 17:45                   ` Jeff King
2019-05-03 18:44                     ` SZEDER Gábor
2019-05-05  5:28                     ` Junio C Hamano
2019-05-05 18:09                       ` Jacob Keller
2019-05-05 18:08                     ` Jacob Keller
2019-05-06  5:11                       ` [PATCH] coccicheck: optionally process every source file at once Jeff King
2019-05-06  9:34                         ` Duy Nguyen
2019-05-06 23:43                           ` [PATCH] coccicheck: optionally batch spatch invocations Jeff King
2019-05-07  1:41                             ` Jacob Keller
2019-05-07  2:04                               ` Jeff King
2019-05-07  2:42                             ` Junio C Hamano
2019-05-07  2:55                               ` Jeff King
2019-05-07  3:04                                 ` Jacob Keller
2019-05-07  4:52                                 ` Junio C Hamano
2019-05-08  7:07                                   ` Jeff King
2019-05-08 12:36                                     ` Denton Liu
2019-05-08 22:39                                       ` Jeff King
2019-05-07 10:20                             ` Duy Nguyen
2019-05-07 11:19                             ` SZEDER Gábor
2021-03-02 20:51                             ` [PATCH] Makefile: fix bugs in coccicheck and speed it up Ævar Arnfjörð Bjarmason
2021-03-03  9:43                               ` Denton Liu
2021-03-03 11:45                               ` Ævar Arnfjörð Bjarmason
2021-03-04 23:18                               ` Junio C Hamano
2021-03-05 11:17                                 ` Ævar Arnfjörð Bjarmason
2021-03-05 10:24                               ` Jeff King
2021-03-05 17:20                                 ` Ævar Arnfjörð Bjarmason
2021-03-06 10:59                                   ` Jeff King
2021-03-05 17:07                               ` [PATCH v2 0/4] Makefile/coccicheck: fix bugs " Ævar Arnfjörð Bjarmason
2021-03-05 19:10                                 ` René Scharfe.
     [not found]                                   ` <xmqqim659u57.fsf@gitster.c.googlers.com>
2021-03-06 11:26                                     ` René Scharfe.
2021-03-06 12:43                                       ` René Scharfe.
     [not found]                                       ` <xmqqft16914r.fsf@gitster.c.googlers.com>
2021-03-13 16:10                                         ` René Scharfe.
2021-03-06 17:27                                   ` Ævar Arnfjörð Bjarmason
2021-03-06 17:41                                     ` René Scharfe.
2021-03-06 17:52                                       ` Ævar Arnfjörð Bjarmason
2021-03-06 19:08                                         ` René Scharfe.
2021-03-05 17:07                               ` [PATCH v2 1/4] Makefile/coccicheck: add comment heading for all SPATCH flags Ævar Arnfjörð Bjarmason
2021-03-05 17:07                               ` [PATCH v2 2/4] Makefile/coccicheck: speed up and fix bug with duplicate hunks Ævar Arnfjörð Bjarmason
2021-03-06 10:45                                 ` Jeff King
2021-03-06 19:29                                   ` Ævar Arnfjörð Bjarmason
2021-03-05 17:07                               ` [PATCH v2 3/4] Makefile/coccicheck: allow for setting xargs concurrency Ævar Arnfjörð Bjarmason
2021-03-06 10:51                                 ` Jeff King
2021-03-05 17:07                               ` [PATCH v2 4/4] Makefile/coccicheck: set SPATCH_BATCH_SIZE to 8 Ævar Arnfjörð Bjarmason
2021-03-06 19:25                                 ` [PATCH v2 5/4] Makefile/coccicheck: use --include-headers-for-types Ævar Arnfjörð Bjarmason
2021-03-18 20:49                                   ` SZEDER Gábor
2021-03-19 10:32                                     ` Ævar Arnfjörð Bjarmason
2021-03-22 12:11                                   ` [PATCH v4 0/4] Makefile/coccicheck: fix bugs and speed it up Ævar Arnfjörð Bjarmason
2021-03-22 12:11                                     ` [PATCH v4 1/4] Makefile/coccicheck: add comment heading for all SPATCH flags Ævar Arnfjörð Bjarmason
2021-03-22 18:04                                       ` René Scharfe.
2021-03-22 12:11                                     ` [PATCH v4 2/4] Makefile/coccicheck: speed up and fix bug with duplicate hunks Ævar Arnfjörð Bjarmason
2021-03-22 18:05                                       ` René Scharfe.
2021-03-24 19:19                                         ` Jeff King
2021-03-22 19:09                                       ` Junio C Hamano
2021-03-22 12:11                                     ` [PATCH v4 3/4] Makefile/coccicheck: allow for setting xargs concurrency Ævar Arnfjörð Bjarmason
2021-03-24 19:26                                       ` Jeff King
2021-03-25  2:29                                         ` Ævar Arnfjörð Bjarmason
2021-03-26  4:11                                           ` Jeff King
2021-03-22 12:11                                     ` [PATCH v4 4/4] Makefile/coccicheck: set SPATCH_BATCH_SIZE to 8 Ævar Arnfjörð Bjarmason
2021-03-22 18:05                                       ` René Scharfe.
2021-03-24 19:27                                         ` Jeff King
2021-03-27 17:43                                     ` [PATCH v4 0/4] Makefile/coccicheck: fix bugs and speed it up Junio C Hamano
2021-03-27 19:46                                       ` Ævar Arnfjörð Bjarmason
2019-05-03  9:40               ` [PATCH v3 0/4] remove extern from function declarations Denton Liu
2019-04-23 23:40   ` Denton Liu [this message]
2019-04-23 23:40     ` [PATCH v4 1/4] *.[ch]: remove extern from function declarations using spatch Denton Liu
2019-04-23 23:40     ` [PATCH v4 2/4] *.[ch]: remove extern from function declarations using sed Denton Liu
2019-04-24  4:56       ` Junio C Hamano
2019-04-25 19:00         ` Denton Liu
2019-04-23 23:40     ` [PATCH v4 3/4] *.[ch]: manually align parameter lists Denton Liu
2019-04-23 23:40     ` [PATCH v4 4/4] cocci: prevent extern function declarations Denton Liu
2019-04-29  8:28     ` [PATCH v5 0/3] *** SUBJECT HERE *** Denton Liu
2019-04-29  8:28       ` [PATCH v5 1/3] *.[ch]: remove extern from function declarations using spatch Denton Liu
2019-04-29  8:28       ` [PATCH v5 2/3] *.[ch]: remove extern from function declarations using sed Denton Liu
2019-04-29  8:28       ` [PATCH v5 3/3] *.[ch]: manually align parameter lists Denton Liu
2019-04-29  8:30       ` [PATCH v5 0/3] *** SUBJECT HERE *** Denton Liu
2019-05-06 11:03         ` Ævar Arnfjörð Bjarmason
2019-05-06 15:34           ` Denton Liu

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

  List information: http://vger.kernel.org/majordomo-info.html

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=cover.1556062365.git.liu.denton@gmail.com \
    --to=liu.denton@gmail.com \
    --cc=emilyshaffer@google.com \
    --cc=git@vger.kernel.org \
    --cc=gitster@pobox.com \
    --cc=peff@peff.net \
    --cc=t.gummerer@gmail.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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).