[PATCH] Re: Create an abstract Repository class
Nicolas Sebrecht
nicolas.s-dev at laposte.net
Wed Mar 2 17:48:21 GMT 2011
On Wed, Mar 02, 2011 at 12:09:27PM +0100, Sebastian Spaeth wrote:
>
> A Repository() returns the correctly instanciated dervivate of a
> BaseRepository, depending on the parameters passed to it. The returned
> instance is eg an ImapRepository(). This makes the code look nicer,
> and we have less functions lying around outside of classes (no more
> global LoadRepository() function).
>
> This will also enable us to conveniently hand back a
> LocalStatusRepository based on SQLITE rather than plain text, if the
> user configures this to be the experimental and optional backend
> (once it exists).
Code looks correct to me even if it should be updated against current
master.
> diff --git a/offlineimap/repository/__init__.py b/offlineimap/repository/__init__.py
> index be5c29e..ea97b27 100644
> --- a/offlineimap/repository/__init__.py
> +++ b/offlineimap/repository/__init__.py
> @@ -1 +1,71 @@
> -__all__ = ['Gmail', 'IMAP', 'Base', 'Maildir', 'LocalStatus']
> +# Copyright (C) 2002-2007 John Goerzen
> +# <jgoerzen at complete.org>
Hugh?
What about
Copyright (C) 2011 Sebastian Spaeth <your at email> and contributors.
?
--
Nicolas Sebrecht
More information about the OfflineIMAP-project
mailing list