[PATCH] Add "extern" qualifier to function declarations in header files

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



This patch prepends "extern " to function declarations in
all header files in the root source directory.

Lines that were longer then 80 characters, are kept this way.
Declarations with more than 80 characters that were stretched
over several lines, are reindented if the indentation followed
some logic.

Signed-off-by: Stephan Beyer <s-beyer@xxxxxxx>
---
Hi,

Junio wrote in <7vod3zuqpv.fsf@xxxxxxxxxxxxxxxxxxxxxxxxxx>:
> > Is this a mistake that some forward declarations in header files are not
> > declared "extern"?
> 
> Yup, that looks old fashioned.

Ok, so I changed this, though I think it's not really useful because it's
really only cosmetic.  (But I needed some very simple task to bring me down
to make me go to bed now.) ;-)

The patch itself is 53kiB, so it's not inline-attached, but can be found
on
	http://pkqs.net/~sbeyer/tmp/extern.patch
or the commit can be fetched from
	git://repo.or.cz/git/sbeyer.git extern

I've grepped through *.h and it seems that no function declaration is left
without either "static inline" or "extern" now.

 attr.h            |    4 +-
 blob.h            |    6 ++--
 branch.h          |    7 +++--
 bundle.h          |   14 +++++-----
 cache-tree.h      |   18 ++++++------
 cache.h           |   50 ++++++++++++++++++------------------
 color.h           |   10 +++---
 commit.h          |   72 ++++++++++++++++++++++++++--------------------------
 diff.h            |    2 +-
 diffcore.h        |    6 ++--
 fetch-pack.h      |    2 +-
 fsck.h            |    6 ++--
 git-compat-util.h |    8 +++---
 graph.h           |   24 +++++++++---------
 list-objects.h    |    4 +-
 ll-merge.h        |   12 ++++----
 log-tree.h        |   20 +++++++-------
 mailmap.h         |    4 +-
 merge-recursive.h |   26 +++++++++---------
 object.h          |   24 +++++++++---------
 pack-refs.h       |    2 +-
 pack-revindex.h   |    2 +-
 patch-ids.h       |    8 +++---
 pkt-line.h        |    8 +++---
 progress.h        |   16 ++++++-----
 quote.h           |    4 +-
 refs.h            |    4 +-
 remote.h          |   51 ++++++++++++++++++++-----------------
 revision.h        |    2 +-
 run-command.h     |   16 ++++++------
 send-pack.h       |    6 ++--
 shortlog.h        |    6 ++--
 sideband.h        |    4 +-
 string-list.h     |   16 ++++++------
 transport.h       |   19 +++++++------
 tree-walk.h       |   12 ++++----
 tree.h            |    8 +++---
 unpack-trees.h    |   10 +++---
 utf8.h            |   12 ++++----
 walker.h          |   15 ++++++-----
 wt-status.h       |    6 ++--
 xdiff-interface.h |   14 +++++-----
 42 files changed, 284 insertions(+), 276 deletions(-)
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Linux Kernel Development]     [Gcc Help]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [V4L]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Fedora Users]

  Powered by Linux