Kedar,
Thanks.
I'm not sure of your view though, you say Absolutely", and "better
not to do it".
I'm trying go figure out what to do. Options seem to be:
1) remove the old get/set methods
2) leave the old get/set methods, but throw an exception
3) leave the old get/set methods, pass them through as before
4) leave the old get/set methods, pass them through to the new names.
Of the above, (1) and (2) are backward-incompatible.
(3) works, but might be confusing.
(4) has only minor drawbacks.
Thoughts?
Lloyd
On May 29, 2007, at 4:37 PM, kedar wrote:
>
>
> Lloyd L Chambers wrote:
>> Kedar,
>>
>> To determine how to proceed with AMX, I need to be clear on the
>> behavior--
>>
>> 1. Can the "old" attributes still be set (changed)?
> Yes, I think so.
>>
>> 2. Are the "old" attributes used at all?
> No, I don't think so.
>>
>> 3. Does it make sense to have AMX "patch through" the old names
>> to the new ones?
>>
> Absolutely. That means more work in AMX layer though. It's better
> not to do it. But it's your call.
>
> Kedar
>> Thanks,
>> Lloyd
>>
>> ----------------
>> Solution: Since we can't "remove" the attributes because that means
>> old XML can't validate against new DTD, we are deprecating these
>> attributes and introducing correctly named attributes as optional
>> ones.
>> We have done a similar thing with security-service element. There is
>> no change in the semantics of the attributes, AFAIK.
>>
>> What is changing: The element http-protocol will be:
>> <!ATTLIST http-protocol
>> version CDATA "HTTP/1.1"
>> dns-lookup-enabled %boolean; "false"
>> forced-type CDATA "text/html; charset=iso-8859-1"
>> default-type CDATA "text/html; charset=iso-8859-1"
>> forced-response-type CDATA "AttributeIsDeprecated"
>> default-response-type CDATA "AttributeIsDeprecated"
>> ssl-enabled %boolean; "true">
>>
>> instead of:
>> <!ATTLIST http-protocol
>> version CDATA "HTTP/1.1"
>> dns-lookup-enabled %boolean; "false"
>> forced-response-type CDATA "text/html; charset=iso-8859-1"
>> default-response-type CDATA "text/html; charset=iso-8859-1"
>> ssl-enabled %boolean; "true">
>>
>>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: admin-unsubscribe_at_glassfish.dev.java.net
> For additional commands, e-mail: admin-help_at_glassfish.dev.java.net
>