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

Reply via email to