On Wed, 2012-06-20 at 18:28 +0200, Stephan Bergmann wrote:
> IIRC, the logic of cppumaker is to first generate temporary .hpp and
> .hdl files and only copy them over existing ones in solver if they are
> actually different. That way, it should easily happen that a .hdl file
> is updated while the corresponding .hpp file is not. (For example, when
Fair enough; but we could presumably easily tweak that such that a .hdl
and .hpp would be updated in lock-step; after all if the .hpp is not
changed - it necessarily includes the .hdl - so we didn't really gain
anything from not touching the .hpp :-)
[ indeed, I'm sure there is one - but what is the benefit of separating
the .hdl files out, if we are sure that each .hpp will include all of
it's dependent .hpp's ? ].
> a method is added to an interface and cppumaker is not running in
> "comprehensive type information" mode; or when a bug in cppumaker is
> fixed affecting .hdl but not .hpp output).
Sure sure - but we could conceptually tweak cppumaker to do the diff on
the pair and update both timestamps if one changes - and loose 25% of
our dependency rules with a trivial regexp :-)
> > Also - do all IDL generated .hpp deps ultimately include all the .hdl
>
> Each .hpp file includes the corresponding .hdl file (if any), yes.
Great.
> > Then again a small number of files include .hdl files directly;
>> is that a bug ? :-)
>
> Looks like a bug, yes. Care to share the list?
Nice; it's easier to generate than send:
git grep 'include.*\.hdl'
only 127 instances luckily.
ATB,
Michael.
--
michael.me...@suse.com <><, Pseudo Engineer, itinerant idiot
_______________________________________________
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice
