Ed Burns wrote:
>>>>>> On Tue, 28 Aug 2007 09:42:08 -0700, Ryan Lubke <Ryan.Lubke_at_Sun.COM> said:
>>>>>>
>
> RL> +
> RL> + /**
> RL> + * <p>Holds the base client ID that will be used to generate per-row
> RL> + * client IDs (this will be null if this UIData is nested within another).</p>
> RL> + */
> RL> + private String baseClientId;
>
> EB> If this is not a part of the saved state, mark it transient using the
> EB> "transient" java keyword.
>
> RL> Why? UIData isn't serializable.
>
> I like using transient to indicate that this ivar is clearly not a part
> of the saved and restored state. Also, just because UIData isn't
> serializable, doesn't mean someone somewhere will try to serialize an
> instance. I think we used to do that in our state saving implementation
> somewhere along the line anyway.
>
>
>
What if I added comments to the javadocs for these ivars stating they
are not part
of the component state instead?