Re: [Guidolib-devel] Code Changes

[Jerome B. <be...@gr...>]

At 17:24 05/02/2004, you wrote:

Hello,

>I realize now why I put gFontDefs in GDeviceDefs.h - it needs the
>GFontRef/GFontInfos class which is defined there.  To include it in
>GUIDOEngine.h would either require moving those classes or adding an
>#include to GUIDOEngine.h, which is undesirable as it would necessitate
>distributing two extra and unnecessary header files with the library.
>Any ideas?

   I had a lot of fun too with those headers. Maybe it's time for the big
spring cleaning. As you just say, it's better to keep separate
"public" and "internal" headers. The game is to have a minimal set
of public headers, minimum dependencies and a fast compile time.

Maybe you could deal with remaining headers that are still not
in /src/include :

/lib-score-engine/defines.h
/parser/guido.h
/parser/GuidoDefs.h

remove some (just as you did with nview.h), maybe create a brand
new one properly placed in /include. Yet another include file is
/include/GUIDOTypes.h

But for now you can let gFontDefs in GDeviceDefs.h if other solutions
cause too many problems.

Cheers,
Jerome