Last minute fixes to the go bindings

Justus Winter 4winter at informatik.uni-hamburg.de
Thu May 10 07:36:50 PDT 2012


Quoting Austin Clements (2012-05-09 20:07:00)
> LGTM.
> 
> Quoth Justus Winter on May 09 at 12:23 pm:
> > Hi everyone :)
> > 
> > this is a small patch series that I'd like to see included in 0.13.
> > 
> > The first patch updates notmuch-addrlookup with respect to Austins
> > recent change of the notmuch_database_open function. This fixes the
> > compilation of the utility.
> > 
> > The second patch fixes the values of all the STATUS_* constants. Turns
> > out all of them were set to zero by accident. This isn't a c style
> > enum, golang has the iota operator to do this kind of stuff. This
> > fixes the error handling for all the users of the go bindings.
> 
> 'Doh!  Is it possible to get the status values out of cgo, rather than
> depending on the list in libnotmuch and the list in Go being in the
> same order?

Yes, it is and I've prepared a patch. But since I'm living in go 1
land here I prepared it on top of my go 1 patchset and I'm thinking of
sending it once that one got merged.

@David: what do you think about merging this patch set?

Justus


More information about the notmuch mailing list