Easyhack: Removing some unused functions

Tor Lillqvist tml at iki.fi
Fri Feb 22 23:07:03 PST 2013


> I'm a new contributor.

Welcome!

> I've attached the patch file which was based
> off of master in the git repo.

Thanks. I am verifying it but it looks fine.

One small detail: Please keep the commit comment lines under 80
columns. Not sure if there is any technical reason for this, but it's
our convention.

> there were an obscene amount of warnings (Mac OSX 10.8.2).

Were those in the bundled 3rd-party libraries (like icu, nss, etc) or
in LibreOffice's own code? Our own code is supposed to be warning-free
with most current compilers (except a few separately handled places,
where we use #pragmas or options to make warnings non-fatal, or go
away). It is a good idea to configure your build with --enable-werror;
then the -Werror flag will be used when compiling LO's own code, and
any warnings will cause the build to stop.

> code is contributed under the LGPLv3+ / MPL

Ah OK. It might be easiest to give a blanket statement about past and
future contributions instead of remembering to say this with each
change,

Also, nowadays it is easier (we hope) to submit and review/accept
changes through gerrit instead of email. See
https://wiki.documentfoundation.org/Development/gerrit . (I hope that
is the relevant wiki page...)

Cheers,
--tml


More information about the LibreOffice mailing list