Hi Ben, sorry for the delay, coming back to that now. > - JGrassUtilities.java (org.geotools.gce.grassraster.JGrassUtilities) looks > like it contains colour tables (names and RGB components) that are directly > converted from MIT X Window System's rgb.txt in its entirety. This should be > acknowledged, and we should accept and include the relevant MIT license in > LICENSE.txt(?) and/or review.apt.
Solved. > - Format.java (org.geotools.gce.grassraster.core.color.Format) > JlsTokenizer.java () has a GeoTools header, but it also has a third-party > header. Should this have just the third-party header? If we have not > substantially contributed to this file, then it should not have a a GeoTools > header, as we can make no claim to it. The third-party header identifies the > author as "The JODD team" and imposes what looks like a three-clause BSD > licence (the good one). This needs to be included in LICENSE.txt(?) and/or > review.apt. Solved > - JlsTokenizer.java (org.geotools.gce.grassraster.core.color.JlsTokenizer) > similarly has a third-party header that it may not need. It is by "JSkeet" > and is identified as being LGPLv2 (or later). (Same as GeoTools, not further > mention required.) Solved > - Only one of the test classes has a GeoTools header. I like headers on > tests, which are also code, but this is rarely enforced. Done > - What is the source of the data in TestMaps.java > (org.geotools.gce.grassraster.TestMaps)? I see no mention. > > - The test data files (not in test-data, I note) are pretty vague, but do > not seem recognisable. Andrea, you might mention their source, if you know. Both the testmaps and data are synthetically generated. I added a note in both the class and LICENSE file. > Good work, Andrea. Looks like, with a few minor review.apt updates, you get > a big tick. Ben, thank you so much for reviewing this. Much appreciated. Best regards, Andrea > > Kind regards, > Ben. > > > On 11/05/11 10:46, Ben Caradoc-Davies wrote: >> >> That's grassraster? Agreed. >> >> On 11/05/11 10:26, Jody Garnett wrote: >>> >>> I will make you a deal; want to review the headers for moovida and I >>> will review for app-schema? >> > > -- > Ben Caradoc-Davies <[email protected]> > Software Engineering Team Leader > CSIRO Earth Science and Resource Engineering > Australian Resources Research Centre > ------------------------------------------------------------------------------ Simplify data backup and recovery for your virtual environment with vRanger. Installation's a snap, and flexible recovery options mean your data is safe, secure and there when you need it. Discover what all the cheering's about. Get your free trial download today. http://p.sf.net/sfu/quest-dev2dev2 _______________________________________________ Geotools-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/geotools-devel
