Re: [ZODB-Dev] An interface for broken objects?

2010-01-08 Thread Hanno Schlichting
On Sun, Jan 3, 2010 at 5:38 PM, Jim Fulton j...@zope.com wrote:
 I suggest including these in the interface.

Thanks for fixing this yourself and merging. Sorry, I didn't get
around to extend things myself.

Hanno
___
For more information about ZODB, see the ZODB Wiki:
http://www.zope.org/Wikis/ZODB/

ZODB-Dev mailing list  -  ZODB-Dev@zope.org
https://mail.zope.org/mailman/listinfo/zodb-dev


Re: [ZODB-Dev] An interface for broken objects?

2010-01-03 Thread Jim Fulton
On Thu, Dec 31, 2009 at 3:02 PM, Hanno Schlichting ha...@hannosch.eu wrote:
 On Thu, Dec 31, 2009 at 6:03 PM, Jim Fulton j...@zope.com wrote:
 On Thu, Dec 31, 2009 at 11:46 AM, Hanno Schlichting ha...@hannosch.eu 
 wrote:
 Is this something that could be put into the ZODB itself and have the
 ZODB.broken.Broken class directly implement this interface?

 +1. although it shouldn't be an empty interface.

 Ok. I've gone ahead and made a branch for this at:

 svn+ssh://svn.zope.org/repos/main/ZODB/branches/hannosch-ibroken

 It's a single changeset: http://svn.zope.org/ZODB/?rev=107467view=rev

Thanks.

 I only specified the custom exception thrown by the __setattr__ in the
 interface. Everything else is private double underscore methods. I
 wasn't sure if any of those should really be specified formally.

Leading *and* trailing double underscores imply special, rather than private.

I suggest including these in the interface.

 Thanks for considering to merge this :)

Thanks for taking the time to do this.

Jim

-- 
Jim Fulton
___
For more information about ZODB, see the ZODB Wiki:
http://www.zope.org/Wikis/ZODB/

ZODB-Dev mailing list  -  ZODB-Dev@zope.org
https://mail.zope.org/mailman/listinfo/zodb-dev


Re: [ZODB-Dev] An interface for broken objects?

2009-12-31 Thread Jim Fulton
On Thu, Dec 31, 2009 at 11:46 AM, Hanno Schlichting ha...@hannosch.eu wrote:
 Hi.

 We currently have a package called zope.broken whose entire real content is:

 import zope.interface

 class IBroken(zope.interface.Interface):
    Marker interface for broken objects
    

 This is used for example by zope.container, which won't try to set
 __name__ and __parent__ pointers on these objects.

 Is this something that could be put into the ZODB itself and have the
 ZODB.broken.Broken class directly implement this interface?

+1. although it shouldn't be an empty interface.

Jim

-- 
Jim Fulton
___
For more information about ZODB, see the ZODB Wiki:
http://www.zope.org/Wikis/ZODB/

ZODB-Dev mailing list  -  ZODB-Dev@zope.org
https://mail.zope.org/mailman/listinfo/zodb-dev


Re: [ZODB-Dev] An interface for broken objects?

2009-12-31 Thread Hanno Schlichting
On Thu, Dec 31, 2009 at 6:03 PM, Jim Fulton j...@zope.com wrote:
 On Thu, Dec 31, 2009 at 11:46 AM, Hanno Schlichting ha...@hannosch.eu wrote:
 Is this something that could be put into the ZODB itself and have the
 ZODB.broken.Broken class directly implement this interface?

 +1. although it shouldn't be an empty interface.

Ok. I've gone ahead and made a branch for this at:

svn+ssh://svn.zope.org/repos/main/ZODB/branches/hannosch-ibroken

It's a single changeset: http://svn.zope.org/ZODB/?rev=107467view=rev

I only specified the custom exception thrown by the __setattr__ in the
interface. Everything else is private double underscore methods. I
wasn't sure if any of those should really be specified formally.

Thanks for considering to merge this :)

Happy new year,
Hanno
___
For more information about ZODB, see the ZODB Wiki:
http://www.zope.org/Wikis/ZODB/

ZODB-Dev mailing list  -  ZODB-Dev@zope.org
https://mail.zope.org/mailman/listinfo/zodb-dev