I've changed this as proposed (I think) and added some tests. 
Having the exception throw directly seems to have the advantage that I get 
consistent behavior between internal and external modules - so that seems to be 
good.

As resolution doesn't seem to happen in the current scenario, the only thing 
that's still not that nice is that we're updating the vector for something 
that's in principle a read-only operation. But, as you said, that's probably a 
minor overhead.
-- 
https://code.launchpad.net/~zorba-coders/zorba/bug-988417-block-internal-module/+merge/103542
Your team Zorba Coders is subscribed to branch lp:zorba.

-- 
Mailing list: https://launchpad.net/~zorba-coders
Post to     : zorba-coders@lists.launchpad.net
Unsubscribe : https://launchpad.net/~zorba-coders
More help   : https://help.launchpad.net/ListHelp

Reply via email to