Menu
▾
▴
Re: [JPP-Devel] org.openjump.* vs com.vividsolutions.* WAS: Duplicate code for WMS
|
From: Stefan S. <ss...@ge...> - 2017-02-05 00:32:58
|
<span style="font-size:10pt;"> <p style="font-size:10pt;margin-top:0;margin-bottom:0;">Hey,</p><p style="font-size:10pt;margin-top:0;margin-bottom:0;"> </p><p style="font-size:10pt;margin-top:0;margin-bottom:0;">well, the decision to split was rather practical as Jump was still under development at the time of the first OJ versions. So this way we would know whats new from our side, not even forgetting that we included stuff from others too with their proper package path (or i did at least) ;)</p><p style="font-size:10pt;margin-top:0;margin-bottom:0;">And yes putting things back together also seems difficult if OJ 1.x should not loose backward compatibility in particular for external plugins - if this its worth, you have decide looking forward.</p><font><p style="font-size:10pt;margin-top:0;margin-bottom:0;"> </p><p style="font-size:10pt;margin-top:0;margin-bottom:0;">so, my 2 cents ;)</p><p style="font-size:10pt;margin-top:0;margin-bottom:0;">stefan from the summer</p><p style="font-size:10pt;margin-top:0;margin-bottom:0;"> </p>------ Originalnachricht ------<br><b>Von: </b>Michaël Michaud <m.m...@or...><br><b>Datum: </b>03.02.2017 14:53<br><b>An: </b>OpenJump develop and use;<br><b>Betreff:</b>Re: [JPP-Devel] org.openjump.* vs com.vividsolutions.* WAS: Duplicate code for WMS<br><br><pre>Hi, Indeed, I regret from the beginning that we have split the code into com.vividsolutions and org.openjump as it is quite difficult to find things. I agree that related code should stay together, but "related" is "vague". If you have more specific rules, we can discuss about it (but a theoretical approach seems difficult). We can also discuss on the list about where to add new classes (practical approach). Michaël Le 02/02/2017 à 16:34, edg...@we... a écrit : > we should probably talk about placing classes in general. i prefer to have code that belongs together in one place. > i don't recall an instance that i placed code under org.openjump , because when i added eg. something to wms and that was placed under com.vividsolutions, the additions went there as well. > > just because we are now OpenJUMP seems not to be a valid reason to make development harder on ourselfs by cluttering packages. > > what's your point of view? ..ede > > > On 02.02.2017 16:26, Giuseppe Aruta wrote: >> I agree with Ede. We didn't had any feedback if anything was working or not. >> com/vividsolutions/jump/workbench/ui/plugin/wms/WMSScaleStylePanel.java >> should probably moved to org/openjump/core/ui/plugin/wms (my fault....) >> >> 2017-02-02 13:45 GMT+01:00 <edg...@we... <mailto:edg...@we...>>: >> >> On 02.02.2017 13 <tel:02.02.2017%2013>:29, Michaël Michaud wrote: >> > AddWMSQueryPlugIn >> > No more used ? >> > >> >> let's keep that for reference. we can deprecate it though. ..ede >> >> ------------------------------------------------------------------------------ >> Check out the vibrant tech community on one of the world's most >> engaging tech sites, SlashDot.org! http://sdm.link/slashdot >> _______________________________________________ >> Jump-pilot-devel mailing list >> Jum...@li... <mailto:Jum...@li...> >> https://lists.sourceforge.net/lists/listinfo/jump-pilot-devel <https://lists.sourceforge.net/lists/listinfo/jump-pilot-devel> >> >> >> >> >> ------------------------------------------------------------------------------ >> Check out the vibrant tech community on one of the world's most >> engaging tech sites, SlashDot.org! http://sdm.link/slashdot >> >> >> >> _______________________________________________ >> Jump-pilot-devel mailing list >> Jum...@li... >> https://lists.sourceforge.net/lists/listinfo/jump-pilot-devel >> > ------------------------------------------------------------------------------ > Check out the vibrant tech community on one of the world's most > engaging tech sites, SlashDot.org! http://sdm.link/slashdot > _______________________________________________ > Jump-pilot-devel mailing list > Jum...@li... > https://lists.sourceforge.net/lists/listinfo/jump-pilot-devel > ------------------------------------------------------------------------------ Check out the vibrant tech community on one of the world's most engaging tech sites, SlashDot.org! http://sdm.link/slashdot _______________________________________________ Jump-pilot-devel mailing list Jum...@li... https://lists.sourceforge.net/lists/listinfo/jump-pilot-devel </pre></span> |