Andre Poenitz <[EMAIL PROTECTED]> writes:

| > This shouldn't leak. And yes, it is a copy of FontTable(pos, font)
| > which is inserted into the fontlist. But the FontTable destructor and
| > LyXfont's destructor should take care of cleaning up in this case.
| > 
| > This could be a but in te vector template.
| 
| I doubt that. If it was a leak in vector, it'd quite probably leak in other
| cases, too. I rather suspect something fishy in the respective items of the
| vector.  I had a look but did not find anything.
| 
| Anyway, I would suggest to change a couple of pointers to references just
| to remove sources of potential problems. E.g. Language could have an
| Encoding _reference_. It is never changed after construction.

Oh, I agree completely. Pointers are evil, references are good.
(as is objects, assignable and copyable)

        Lgb

Reply via email to