Jean-Marc Lasgouttes wrote:
> Juergen, if seems to me that the following code
> +void InsetInclude::updateBibfilesCache(Buffer const & buffer)
> +{
> + if (loadIfNeeded(buffer, params_)) {
> [...]
> + }
> +}
> +
> +std::vector<string> const &
> +InsetInclude::getBibfilesCache(Buffer const & buffer) const
> +{
> + if (loadIfNeeded(buffer, params_)) {
> [...]
> + }
> triggers a load of child document everytime the master document is
> loaded. This is bad, especially since this behaviour depends on
> whether natbib is used or not.
That's bad indeed. I think we should add another bool IsLoaded that checks if
the file is loaded but does not load it.
Georg, can you rephrase why exactly the insetinclude changes where needed?
Would the proposed solution still work?
> I think this is what triggers Georg observation that the session patch
> loads child documents twice.
>
> We have to clarify this issue for 1.4.2, IMO.
Yes.
Jürgen
