On 01/04/2012 06:01 AM, Lubos Lunak wrote:
On Saturday 31 of December 2011, Brad Sowden wrote:
On 12/29/2011 10:26 PM, Brad Sowden wrote:
Hi,

See attached.

Actually, this patch causes "make dev-install -o build" to fail. I have
no idea why as the patch looks extremely innocent.

If I drop this patch (and patch 8 which removes the SvStrings
definition) then everything else applies correctly and "make dev-install
-o build" succeeds.

-    for(i = 0; i<  aFldNames.Count();i++)
+    for(std::vector<String*>::const_iterator it(aFldNames.begin()); it !=
aFldNames.end(); ++it)
      {
-        pArray[i] = *aFldNames.GetObject(i);
+        pArray[i] = **it;

                ^^^

+        delete *it;
      }

  Fixed, pushed.

Doh! Sorry and thanks for the push.

Brad
_______________________________________________
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice

Reply via email to