[Bug 90222] replace ScaList in scaddins with a std container

bugzilla-daemon at bugs.documentfoundation.org bugzilla-daemon at bugs.documentfoundation.org
Thu Mar 26 05:05:31 PDT 2015


https://bugs.documentfoundation.org/show_bug.cgi?id=90222

--- Comment #5 from David Tardon <dtardon at redhat.com> ---
(In reply to Pieter Adriaensen from comment #4)
> I still have a question though: The ScaList is always used to put data of
> the same class in. So isn't it better to change it to vector<T*> instead of
> being a vector<void*> and doing static_casts?

Of course it is better! I assumed this is something that does not have to be
explicitly mentioned... Also, it should be vector<T>, if possible.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.freedesktop.org/archives/libreoffice/attachments/20150326/f22bcfde/attachment.html>


More information about the LibreOffice mailing list