[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Orekit Developers] Warnings cleanup



Yes, those are it.  I had another one due to the way Eclipse setup the project from the Maven file and another in a unit test that I just committed.  

On Fri, Oct 3, 2014 at 3:46 AM, MAISONOBE Luc <luc.maisonobe@c-s.fr> wrote:
Hank Grabowski <hank@applieddefense.com> a écrit :

Hello all,

Would it be okay if I clean up the current list of compiler warnings?
There are currently seven or so in the current head of master.

Sure. The only ones that you should not touch yet are the TODO checkstyle warnings (not compiler warnings) in DSST. A lot of work has been done on DSST but is for now still in the dsst-short-periodics branch. The branch will be merged to master very soon now.

I currently only see 4 compiler warnings (using Eclipse Luna compiler), due to resources not closed (one in SP3Parser, three in OEMParser). What are the other warnings you see?

best regards,
Luc


Hank




----------------------------------------------------------------
This message was sent using IMP, the Internet Messaging Program.