On 21.12.2014 17:07, Michael Michaud wrote: > Hey Ede, > > You made a big change on AttributeType class last thursday, > It's a very interesting change with a lot of potential, but my > concern is how much code it will break, and how much time > it will take to upgrade all the code depending on this class.
valid concern > New data types you inroduced are already available in Edit > Schema plugin, but many drivers/plugin are not ready to > manage these new types. that was my intention and we have to start somewhere. the commit as such does not break a thing. when people start using the new attribute types and trun into errors we can start fixing them. > From my point of view, this is the kind of change which may > justify to create a new branch not again ;).. we are simply too few devs to maintain anymore than one proper branch >as it may break current code which code do you have in mind? > for an undetermined laps of time. shouldn't it break nothing? the "old" attribute types will continue to work as good as before. readers will probably still use the old datatypes and _only_ if the user switches the change will become effective, non? > > What do you think ? Is it an intentional commit ? yes. a first step, nothing more, nothing less. > Are you aware of all the dependencies ? the topic comes up from time to time, but we shy away from it. so why not start? do you have an issue at hand, that breaks OJ's general usability because of this commit? ..ede ------------------------------------------------------------------------------ Download BIRT iHub F-Type - The Free Enterprise-Grade BIRT Server from Actuate! Instantly Supercharge Your Business Reports and Dashboards with Interactivity, Sharing, Native Excel Exports, App Integration & more Get technology previously reserved for billion-dollar corporations, FREE http://pubads.g.doubleclick.net/gampad/clk?id=164703151&iu=/4140/ostg.clktrk _______________________________________________ Jump-pilot-devel mailing list Jump-pilot-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/jump-pilot-devel