[Libreoffice] [PATCH] Replace List with std::vector<>

Joseph Powers jpowers27 at cox.net
Sun Jun 19 21:48:39 PDT 2011


I'm sending the patch for review; not because it needs a lot but just to have someone verify that the memory was leaking and needs to be fixed.

Current fixes:
1. XPropertyList::Clear() looks like it has a memory leek; so I fixed it.

Future Fixes:
1.   List* pBmpList; Should be replaced with a std::vector< Bitmap*>
2. I thinking that once pBmpList is a vector<> I should cleanup the constructors that pass initial size and resize values.
3. I'm noticing that several methods have extra parameters that aren't used. Should I include a patch to clean them up also?

Thanks,
Joe P.

-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-Replace-List-with-std-vector-XPropertyEntry.patch
Type: application/octet-stream
Size: 16213 bytes
Desc: not available
URL: <http://lists.freedesktop.org/archives/libreoffice/attachments/20110619/0580ec3c/attachment-0001.obj>


More information about the LibreOffice mailing list