Should functions be made "extern" in header files?

Should functions be made extern in header files? Or are they extern by default?

For example, should I write this:

// birthdays.h
struct person find_birthday(const char* name);

or this:

// birthdays.h
extern struct person find_birthday(const char* name);

Solution 1:

From The C Book:

If a declaration contains the extern storage class specifier, or is the declaration of a function with no storage class specifier (or both), then:

• If there is already a visible declaration of that identifier with file scope, the resulting linkage is the same as that of the visible declaration;
• otherwise the result is external linkage.

So if this is the only time it's declared in the translation unit, it will have external linkage.

Solution 2:

They are implicitly declared with "extern".

Solution 3:

Functions declared in headers are normally (unless you work really hard) extern. Personally, I prefer to see the explicit keyword there - but the compiler doesn't need it. It reminds the readers that they are extern, and since humans are more fallible than computers, I find the reminder helps.

With variables, it is important to use the extern keyword (and no initializer) in the header file. Consequently, for symmetry with the (very few) global variables declared in headers, I use extern with the function too - even though it is strictly not necessary.