Andre Poenitz wrote: > On Tue, Jun 03, 2003 at 06:17:10PM +0000, Angus Leeming wrote: >> This cleans up InsetGraphics::Cache and renames it as GraphicsInset so >> that the external inset can use it too. > > What do you need old_ascent_ for? > > + old_ascent_ = (type == IMAGE) ? loader_.image()->getHeight() : 50; > + > + dim.asc = old_ascent_; > + dim.des = 0; > + [...] > + dim_ = dim; > > You could use dim_.asc instead...
This is what I call "Juergen magic". I didn't really understand it then. I still don't. If you tell me that it is "premature optimization" then I will remove it. -- Angus