> > This is wrong. Because buffer_->embeddedFiles(), unless an update() is > > called, > > > updateBibfilesCache() calls EmbeddedFileList::enable(), which calls > update(). This is what the other question was about, really. > > + } > The vector<FileName> tells us what files we have, and then we look up > their EmbeddedFile objects to find out what needs copying. Those are > up-to-date for the reasons mentioned above. > > But I agree with you that maybe this could be better done elsewhere.
Your code might work, with some effort, but it is not how embedding is supposed to work. It would be logically much simpler if each inset takes care of its own embedded files. Cheers, Bo
