Hi all,
The attribute “charset” in ENCAPSULATED should probably be renamed to “encoding” to be consistent with similar attributes in DV_TEXT etc. Both “charset” and “encoding” are using the same code set “character sets” from openEHR terminology.
Cheers,
Rong
Rong Chen wrote:
Hi all,
The attribute "charset" in ENCAPSULATED should probably be renamed to
"encoding" to be consistent with similar attributes in DV_TEXT etc.
Both "charset" and "encoding" are using the same code set "character
sets" from openEHR terminology.
Theoretically this is of course true. We didn't do it in previous
releases (1.0 and 1.0.1) because it is a change rather than an addition.
I believe we need a default policy of not changing attribute names
without ensuring that all software implementers agree with it, since it
will require software changes, XML schema changes and it will invalidate
any existing DV_ENCAPSULATED data instances (but there may not be many
or any yet). So we have to be careful with changes like this.
The mechanism we have for this is the implementers list. I have
accordingly posted Rong's message there to create a thread allowing
implementers to have their say. If there are no strong objections we can
raise a CR. There may however be objections, and we may have to stick
with the name as it is for this reason....
- thomas beale