On 3/28/07, Steve Brewin <[EMAIL PROTECTED]> wrote:
robert burrell donkin wrote:
<snip>
The key thing is that the original API did not allow the passing of any context information and you have fixed this, which is great. Maybe if I had been making these changes, having realised that we need to pass contextual information around, I might have amended the api slightly differently and added an extra level of indirection. I might have added a context class which contains your ScriptCoordinate and passed this around. In the future, we might find a need to add other contextual information. We could add this without further reworking of the main API by specializing this context class. In this way we could add functionality without breaking code relying on less specialized versions.
+1 unless anyone objects, i'll commit a reworked version which uses a context (for the API calls) - robert --------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
