Hi Tom,
After a short investigation, I found out that the same applies to all
the IDL keywords.
So, "interface" is just a particular case (use String,Module,Sequence,
etc.) .
I would not fix such a big issue without first carefully analyzing the
implications of such a change.
Regards,
Sorin
[EMAIL PROTECTED] wrote:
Hi,
some time ago we migrated to MICO 2.3.11 and recently we discovered some
minor problems with the IDL compiler in this version of MICO. The
following statement is copied from the changelog for 2.3.11:
- fix IDL compiler to report error when identifier collides with IDL keyword
Obviously this is a good fix, but I'm wondering whether there's really
any reason for the check to be case insensitive. In 2.3.11 I cannot use
the identifier '*Interface*' (<- capital 'I') for e.g. an enumerator
because the check introduced matches this to the IDL keyword
'*interface*' (<- no capital).
This change seems to have been introduced in the ParseNode::setIdent
method (idl/parsenode.cc) which now checks whether an identifier matches
a keyword and reports an error if it does. To check this the
KeywordMap::keyword method (idl/keymap.cc) is used. The keyword method
is the one transforming all characters in the identifier to their lower
case representations.
So to my actual questions:
Is there any reason based on the CORBA standard that requires this
matching to be case /insensitive/?
Do we risk breaking other functionality in MICO by changing this
matching to be case /sensitive?/
Is this a potential fix for the next MICO version (making the check case
/sensitive /and only report error when actual matches are made)?
Regards,
Tom
--
Tom Wiggo Sørlikåseth
------------------------------------------------------------------------
_______________________________________________
Mico-devel mailing list
Mico-devel@mico.org
http://www.mico.org/mailman/listinfo/mico-devel
--
Best regards,
Sorin Mustaca
http://www.mustaca.de
_______________________________________________
Mico-devel mailing list
Mico-devel@mico.org
http://www.mico.org/mailman/listinfo/mico-devel
