cayenne-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Michael Gentry <mgen...@masslight.net>
Subject Re: Serialization warnings
Date Thu, 14 Jun 2012 14:58:06 GMT
Ah.  Well, if you leave serialVersionUID out of either class, you get
a warning in that class.  Same applies to the
@SuppressWarnings("serial") annotation.  From the perspective of
reducing warning messages, it needs to be in both classes from what I
can tell.

Thanks,

mrg


On Thu, Jun 14, 2012 at 10:53 AM, Mike Kienenberger <mkienenb@gmail.com> wrote:
> Right.  I understand that.  I just don't remember if it's appropriate
> to specify a serialVersionUID for the subclasses.   "Undecided"
> probably was the wrong word in my original message -- more like "don't
> know"
>
> On Thu, Jun 14, 2012 at 10:36 AM, Michael Gentry <mgentry@masslight.net> wrote:
>> The subclasses would still only be generated once.  I have no desire
>> to change that aspect.
>>
>> Thanks,
>>
>> mrg
>>
>>
>> On Thu, Jun 14, 2012 at 10:14 AM, Mike Kienenberger <mkienenb@gmail.com> wrote:
>>> +1 for doing this for the generate-always classes.
>>>
>>> Undecided about the generate-once subclasses.
>>>
>>> On Thu, Jun 14, 2012 at 10:06 AM, Michael Gentry <mgentry@masslight.net>
wrote:
>>>> Going back to https://issues.apache.org/jira/browse/CAY-1622 ...
>>>>
>>>> How about if Cayenne Modeler has an option for the following choices:
>>>>
>>>> A) Generate @SuppressWarnings("serial") in the super/sub classes.
>>>>
>>>> B) Generate a a calculated serialVersionUID in the superclass and
>>>> default the value in the subclass:
>>>>
>>>> _Foo: protected static final long serialVersionUID = [calculated];
>>>> Foo: protected static final long serialVersionUID = _Foo.serialVersionUID;
>>>>
>>>> This way the superclass will get a new value should the attributes
>>>> change and the default value for the subclass will be the same as the
>>>> superclass, but the developer can override this if they wish.
>>>>
>>>> Thoughts?
>>>>
>>>> mrg

Mime
View raw message