- Added a test to the Slice parser to complain if an operation on a local

    interface or class has an exception specification.
- Added a test to the property parsing code to print a warning on stderr if
    a property is not recognized. This prevents silly typos, such as
    "Ice.config=MyFile" (instead of "Ice.Config=MyFile") from slipping
    through undetected.

1 files changed, 15 insertions, 0 deletions

diff --git a/cpp/src/Slice/Parser.cpp b/cpp/src/Slice/Parser.cpp
index b546bd10445..afb68d357ae 100644
--- a/cpp/src/Slice/Parser.cpp
+++ b/cpp/src/Slice/Parser.cpp
@@ -3620,6 +3620,21 @@ Slice::Operation::setExceptionList(const ExceptionList& el)
     _throws = el;
 
     //
+    // Local interfaces and classes cannot have an exception specification.
+    //
+    if(el.size() != 0) {
+	ClassDefPtr parent = ClassDefPtr::dynamicCast(container());
+	assert(parent);
+	if(parent->isLocal())
+	{
+	    string msg = "operation `" + name() + "' cannot have a throws clause because it is an operation on a ";
+	    msg += parent->kindOf();
+	    _unit->error(msg);
+	    return;
+	}
+    }
+
+    //
     // Check that no exception occurs more than once in the throws clause
     //
     ExceptionList uniqueExceptions = el;