> I'm fairly convinced that we should not be using the
> EmbeddedFiles stuff to represent the parameters here. That just seems
> like mixing up the wrong things.
Why not? We need to store filenames (I know I need to keep the
original filenames), and the embedding status. Using a EmbeddedFiles
object is natural to me.
Because then you have to keep this different representation sync'd with
the one on which InsetCommandParams::write and ::read rely. Which was
the original problem. And we don't need to know all the embedding stuff
here, just two things: the bibfile, and the inzipName.
rh