[PATCH] fdo#39749: calc unit test writing
Markus Mohrhard
markus.mohrhard at googlemail.com
Sat Mar 31 09:15:17 PDT 2012
Hey,
> I'm submitting a patch for the easy hack 39749. I have written one test that
> tests the setbackgroundcolor function in ScDocFunc class
>
Great to have you working on this, I have just some small comments:
- Could you please build with your patch before submitting? The patch
was in several ways broken and I had to fix it to get it build.
- Can you set your git settings so that it sets the author name
correctly. I adjusted it in your patch.
- You still had trailing whitespaces in your commit. Can you try to
set-up your editor to remove them or remove them when our git hook
complains about them?
Except for these smaller problems the patch was good and I pushed it
with my small modifications.
http://cgit.freedesktop.org/libreoffice/core/commit/?id=3d86e4e7223b15d561d1efdac736e932ea41e6a0
Can you confirm that your patch is under MPL/LGPLv3+?
Regards,
Markus
More information about the LibreOffice
mailing list