[OPEN-ILS-DEV] PATCH: srfsh.c (eliminating header)

Mike Rylander mrylander at gmail.com
Mon Jul 9 09:14:08 EDT 2007


On 7/8/07, Scott McKellar <mck9 at swbell.net> wrote:
> One of the things I have been doing in my patches is to move things
> out of header files and into implementation files (i.e. *.c files),
> whenever those things are referenced only in the one implementation
> file.  So far, Mike and Bill haven't complained much.
>
> Tonight's patches carry this policy to its logical conclusion.  I
> moved the entire contents of srfsh.h into srfsh.c.  No other file
> #includes this header, nor is it likely that any ever would.
> Consequently I see no reason to isolate these things in a separate
> file.

The trend continues.  Applied with thanks.

--miker


More information about the Open-ils-dev mailing list