The definition in the header file will cause most calls to the function to be inlined. If any uses of the function remain, they will refer to the single copy in the library. Try to use ANSI C's enum feature when defining lists of constants of the same type. Alastair likes to use stgCast instead of C syntax. He claims it's easier to write and easier to grep for.
Long lines are hard to read, and a sign that the code needs to be restructured anyway.
Add this to your .emacs file. CVS issues Don't be tempted to reindent or reorganise large chunks of code - it generates large diffs in which it's hard to see whether anything else was changed. If you must reindent or reorganise, don't do anything else in that commit and give advance warning that you're about to do it in case anyone else is changing that file.