[Libreoffice] [PUSHED][PATCH] Refactoring for WMF Loading
caolanm at redhat.com
Wed Oct 5 06:03:22 PDT 2011
On Tue, 2011-10-04 at 14:04 +0530, Marc-André Laverdière wrote:
> Based on Caolan's comments, here's another attempt at doing things right.
> I'm unable to test right now (some symlink problem)... please don't push
> right away :)
This looks right to me, pushed now, thanks for this.
Only micro-nit is can leave an argument unnamed rather than self-assign
it to avoid an unused parameter warnings, e.g.
void foo(int a, int, int c)
void foo(int a, int b, int c)
b = b;
More information about the LibreOffice