Re: [Quantlib-dev] [QuantLib-svn] SF.net SVN: quantlib: [14226] trunk/QuantLib

[Ferdinando A. <na...@am...>]

On Thu, Jan 31, 2008 at 10:13 AM, Luigi Ballabio
<lui...@gm...> wrote:
>  >     trunk/QuantLib/ql/instruments/makeswaption.cpp
>  >     trunk/QuantLib/ql/instruments/makeswaption.hpp
> [...]
>  How about putting MakeSwaption together with Swaption, instead? (And the
>  same for the other helpers, of course.)

oh boy! as it often happens, I was thinking in the exact opposite direction ;-)
I was considering to move the MakeSchedule code out of schedule.*pp
into its own files.

The rational for my preference is that we have huge nested
dependencies because of too-large too-often-included hpp files and
this triggers a lot of avoidable recompilations.
While some recompilation is now avoided thanks to the increasing usage
of forward declaration, I would favour breaking up hpp declaration
files in reasonably small units.

By the way, the _whole_ library is currently being recompiled as I
write just because I've added 2 inspectors to the Option class. This
happens just because the enum Type { Put, Call } is inside the Option
class definition instead of being outside (in its own file) and this
enumeration is of course used in a lot of places, even where the
Option class is not used at all :-(

ciao -- Nando