Re: [PATCH 3/7] py3k: All strings are unicode strings in py3k

Subject: Re: [PATCH 3/7] py3k: All strings are unicode strings in py3k

Date: Wed, 14 Dec 2011 14:17:41 +0200

To: Justus Winter, notmuch@notmuchmail.org

Cc:

From: Tomi Ollila


On Wed, 14 Dec 2011 11:58:21 +0100, Justus Winter <4winter@informatik.uni-hamburg.de> wrote:
> ---
>  bindings/python/notmuch/globals.py |    4 ++--
>  1 files changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/bindings/python/notmuch/globals.py b/bindings/python/notmuch/globals.py
> index 54a49b2..99e6a10 100644
> --- a/bindings/python/notmuch/globals.py
> +++ b/bindings/python/notmuch/globals.py
> @@ -51,7 +51,7 @@ class Status(Enum):
>          """Get a (unicode) string representation of a notmuch_status_t value."""
>          # define strings for custom error messages
>          if status == STATUS.NOT_INITIALIZED:
> -            return u"Operation on uninitialized object impossible."
> +            return "Operation on uninitialized object impossible."
>          return unicode(Status._status2str(status))
>  
>  STATUS = Status(['SUCCESS',
> @@ -142,7 +142,7 @@ class NotmuchError(Exception):
>          elif self.status is not None:
>              return STATUS.status2str(self.status)
>          else:
> -            return u'Unknown error'
> +            return 'Unknown error'

Is this u -prefix unnecessary in python 2 too ? Grepping
'u"' and "u'" in **/*.py in puthon bindings source resulted
some more u-prefixed strings in docs/source/conf.py. Should
these be changed in some future patch ?

>  # List of Subclassed exceptions that correspond to STATUS values and are
> -- 
> 1.7.7.3

Tomi

Thread: