> 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

Reply via email to