616 2005-07-29 16:05:09 +0000 Error with shadowed enum declarations 2005-10-03 22:43:43 +0000 1 1 1 Unclassified JacORB IDL compiler 2.2.2 All All RESOLVED FIXED http://lists.spline.inf.fu-berlin.de/pipermail/jacorb-developer/2005-July/007472.html P2 major --- 1 d.griffin gerald.brose Thomas.Leineweber oldest_to_newest 1483 0 d.griffin 2005-07-29 16:05:09 +0000 As reported by John Fisher, the IDL below does not work. It looks like there is a problem in src/org/jacorb/idl/NameTable.java relating to shadowed enums. I've attached a patch that seems to fix the problem. interface A { enum my_enum { ENUM1, ENUM2 }; }; interface B : A { enum my_enum { ENUM1, ENUM3 }; }; 1484 1 163 d.griffin 2005-07-29 16:05:48 +0000 Created attachment 163 Patch to prevent shadowed enums from crashing the IDL compiler 1541 2 Thomas.Leineweber 2005-08-24 12:34:34 +0000 the given patch seems to fix bug #470 and bug #517, too. 1564 3 spiegel 2005-10-03 22:43:43 +0000 I installed Duane's patch. Will also close bug 470 and bug 517. Thanks, Duane and Thomas. 163 2005-07-29 16:05:48 +0000 2005-07-29 16:05:48 +0000 Patch to prevent shadowed enums from crashing the IDL compiler ShadowEnums.patch text/plain 1648 d.griffin SW5kZXg6IHNyYy9vcmcvamFjb3JiL2lkbC9OYW1lVGFibGUuamF2YQo9PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09ClJDUyBm aWxlOiAvY3Zzcm9vdC9qYWNvcmIvSmFjT1JCL3NyYy9vcmcvamFjb3JiL2lkbC9OYW1lVGFibGUu amF2YSx2CnJldHJpZXZpbmcgcmV2aXNpb24gMS4yMwpkaWZmIC11IC1yMS4yMyBOYW1lVGFibGUu amF2YQotLS0gc3JjL29yZy9qYWNvcmIvaWRsL05hbWVUYWJsZS5qYXZhCTI4IE1hciAyMDA1IDE5 OjU4OjI5IC0wMDAwCTEuMjMKKysrIHNyYy9vcmcvamFjb3JiL2lkbC9OYW1lVGFibGUuamF2YQky OSBKdWwgMjAwNSAxMTozNDoyNyAtMDAwMApAQCAtMTcxLDEwICsxNzEsMTMgQEAKICAgICAgICAg ICAgICAgICBzaGFkb3dzLnJlbW92ZSggbmFtZSApOwogICAgICAgICAgICAgICAgIGgucmVtb3Zl KCBuYW1lICk7CiAKLSAgICAgICAgICAgICAgICAvLyByZW1vdmUgdGhlIGluaGVyaXRlZCB0eXBl IGRlZmluaXRpb24sIGEgbmV3IG9uZSB3aWxsIGJlCi0gICAgICAgICAgICAgICAgLy8gYWRkZWQg c29vbiB1bmRlciB0aGlzIG5hbWUhIEFkZGl0aW9uIG9mIHRoaXMgbGluZSBmaXhlcwotICAgICAg ICAgICAgICAgIC8vIGJ1ZyAjMzQ1Ci0gICAgICAgICAgICAgICAgVHlwZU1hcC5yZW1vdmVEZWZp bml0aW9uKCBuYW1lICk7CisgICAgICAgICAgICAgICAgaWYoIGtpbmQuc3RhcnRzV2l0aCggInR5 cGUiICkgKQorICAgICAgICAgICAgICAgIHsKKyAgICAgICAgICAgICAgICAgICAgLy8gcmVtb3Zl IHRoZSBpbmhlcml0ZWQgdHlwZSBkZWZpbml0aW9uLCBhIG5ldyBvbmUgd2lsbCBiZQorICAgICAg ICAgICAgICAgICAgICAvLyBhZGRlZCBzb29uIHVuZGVyIHRoaXMgbmFtZSEgQWRkaXRpb24gb2Yg dGhpcyBsaW5lIGZpeGVzCisgICAgICAgICAgICAgICAgICAgIC8vIGJ1ZyAjMzQ1CisgICAgICAg ICAgICAgICAgICAgIFR5cGVNYXAucmVtb3ZlRGVmaW5pdGlvbiggbmFtZSApOworICAgICAgICAg ICAgICAgIH0KICAgICAgICAgICAgIH0KICAgICAgICAgfQogCkBAIC0zNTQsNiArMzU3LDEzIEBA CiAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAg ICAgICBrZXkuc3Vic3RyaW5nKCBrZXkubGFzdEluZGV4T2YoICcuJyApICksCiAgICAgICAgICAg ICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICBhbmMgKTsK ICAgICAgICAgICAgICAgICAgICAgfQorICAgICAgICAgICAgICAgICAgICBlbHNlCisgICAgICAg ICAgICAgICAgICAgIHsKKyAgICAgICAgICAgICAgICAgICAgICAgIGlmKCBsb2dnZXIuaXNEZWJ1 Z0VuYWJsZWQoKSApIHsKKyAgICAgICAgICAgICAgICAgICAgICAgICAgICBsb2dnZXIuZGVidWco Ii0gTmFtZVRhYmxlLmluaGVyaXQgIiArIGtpbmQgKyAiIGZyb206ICAiICsga2V5KTsKKyAgICAg ICAgICAgICAgICAgICAgICAgIH0KKyAgICAgICAgICAgICAgICAgICAgfQorCiAKICAgICAgICAg ICAgICAgICAgICAgaWYoICFpc0RlZmluZWQoIGtleSApICkKICAgICAgICAgICAgICAgICAgICAg ICAgIHRocm93IG5ldyBSdW50aW1lRXhjZXB0aW9uKCAiQ29tcGlsZXJFcnJvciEiICk7Cg==