Ede:
Let me see if I can provide a little more information.
I'm proposing the addition of a few utility methods to the Feature and
FeatureSchema classes/interfaces. This include methods to clone, compare,
and convert the objects to Strings. These are all pretty standard Java
utility methods that I try to implement on almost all of my classes.
None of these new methods break the current OpenJUMP API.
Having said that, none of these methods are essential, and they don't fix
any bugs. I implemented them because they made unit testing easier.
I'm also proposing to add adaptations of my JUnit text cases for the
Feature and FeatureSchema classes/interfaces. The reason for this is
simple: We don't do very much unit testing in the OpenJUMP core, and we
should. (This is just a first baby step in this direction. I'd like to
continue committing unit tests for the OpenJUMP core as I can develop as
part of the code I write.)
Neither the utility methods or the unit tests are critical changes. I'm
just trying to be a good citizen by pushing some of my improvements from
JUMP-Lib back upstream to OpenJUMP.
No hard feelings if the community decides these changes aren't
improvements. :]
Thanks.
Landon
On Thu, Jun 27, 2013 at 1:36 AM, <edgar.sol...@web.de> wrote:
> On 27.06.2013 05:15, Landon Blake wrote:
> > I'd like to port some features from JUMP-Lib to OpenJUMP. These features
> are listed here:
> >
> >
> http://openjumpblog.wordpress.com/2013/06/27/improvements-from-jump-lib-for-openjump/
> >
> > What do you guys think?
> >
> > I'm basically adding some standard utility methods to the
> Feature/BasicFeature and FeatureSchema. I'd also like to add some JUnit
> tests for Feature/BasicFeature and FeatureSchema to a testing package.
> >
> > If there are no strong objections, I can start the work of porting the
> features one at a time, and post here after each commit so the other
> programmers can review the changes.
> >
>
> could you describe what the advantages of these changes mentioned on the
> blog page are? point by point?
>
> ..ede
>
>
> ------------------------------------------------------------------------------
> This SF.net email is sponsored by Windows:
>
> Build for Windows Store.
>
> http://p.sf.net/sfu/windows-dev2dev
> _______________________________________________
> Jump-pilot-devel mailing list
> Jump-pilot-devel@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/jump-pilot-devel
>
------------------------------------------------------------------------------
This SF.net email is sponsored by Windows:
Build for Windows Store.
http://p.sf.net/sfu/windows-dev2dev
_______________________________________________
Jump-pilot-devel mailing list
Jump-pilot-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/jump-pilot-devel
