Ticket #78: ticket78.patch

Source/Statements.cobra

@@ -912 +912 @@
                     .throwError('Expecting an event to raise.')
             else if expr inherits IdentifierExpr  # ex: raise _eventName
                 _name = expr.name
-                assert expr.potentialType
                 _eventType = expr.potentialType
+                if not _eventType
+                    hint = ''
+                    if expr.type inherits MethodSig 
+                        hint = 'Did you mean to use "[_name](args)" to invoke the methodSig Identifier (delegate)?'
+                    .throwError('The identifier target of "raise" is not an event. [hint]')
                 if _eventType.isDescendantOf(.compiler.delegateType)
                     pass
                 else if _eventType.isDescendantOf(.compiler.exceptionType)

Tests/820-errors/300-statements/320-raise-not-event.cobra

@@ +1 @@
+class Test
+
+    sig SimpleMethod
+
+    def foo(s as SimpleMethod)
+        raise s  # .error. invoke the methodSig Identifier
+        
+    def foo2(s as SimpleMethod)
+        f=10
+        raise f # .error. target of "raise" is not an event
+        
+    def foo3(s as SimpleMethod)
+        s()
+
+    def y
+        print 'Fired y'
+    
+    def main is shared
+        x = Test()
+        x.foo( ref x.y)

Developer/IntermediateReleaseNotes.text

@@ -178 +178 @@
 
 * Added keyword `each`, reserved for future use. Added keywords for various modifiers that were already in use: `abstract`, `extern`, `fake`, `internal`, `new`, `nonvirtual`, `override`, `partial`, `public`, `private`, `protected`, `virtual`. Deprecated `fake` in favor of new `extern`.
 
+* Fixed: Raise on a delegate should generate error message. ticket:78.
+
 * Fixed: The Cobra command line compiler throws an exception for files with an all capital extension (FOO.COBRA).
 
 * Fixed: Using "continue" in new style numeric for loops does not increment the loop variable. ticket:9