namgk / ambienttalk

Automatically exported from code.google.com/p/ambienttalk
0 stars 0 forks source link

uniform access principle does not seem to hold for cancel() method on subscription/publication objects #49

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago
What steps will reproduce the problem?
1. Change line 111 of the provided file to call pub.cancel and sub.cancel 
instead of pub.cancel() and sub.cancel()
2. Run unit tests of the provided file.
3.

What is the expected output? What do you see instead?
Publication/subscription are not cancelled, so the unit test discover objects 
from a previous test and fails.

Please use labels and text to provide additional information.

Original issue reported on code.google.com by egonzal...@gmail.com on 30 Mar 2011 at 3:46

Attachments:

GoogleCodeExporter commented 8 years ago
The problem is that these cancel methods are represented as fields referring to 
closures, e.g.
def cancel := { ... }

A patch is pending that implements the cancel methods of all the 
subscription/publication objects as actual methods (I'm getting unexpected 
exceptions for the moment)

Original comment by tvcut...@gmail.com on 6 Apr 2011 at 1:53

GoogleCodeExporter commented 8 years ago
Fixed in trunk r2779
cancel is now implemented as a method instead of as a field.

Original comment by tvcut...@gmail.com on 8 Apr 2011 at 7:32