incubator-odf-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Rob Weir <>
Subject Re: OdfPackage not throwing an exception on I/O errors
Date Thu, 22 Mar 2012 19:45:22 GMT
On Thu, Mar 22, 2012 at 1:34 PM, Jeremias Maerki <> wrote:
> Hey guys,
> is there a special reason why, String)
> catches all IOExceptions, logs them and just continues? While I can see
> the problems in the logs, my code thinks everything is fine and
> continues normally.

I can't think of a good reason.

> Furthermore, the public save() methods all throw Exception which is
> rather bad style.

I assume you mean that they should limit themselves to IOException?

> Can we please change that? I'll offer to write a proposal in form of a
> patch.

If you can contribute a patch, that would be great.  But please do a
complete rebuild, to make sure the unit tests still work.

So from the top directory:

mvn clean install -Ppedantic



> Thanks,
> Jeremias Maerki

View raw message