From: David Bremner <david@tethera.net>
To: Trevor Jim <tjim@mac.com>, Tomi Ollila <tomi.ollila@iki.fi>,
	notmuch@notmuchmail.org
Subject: Re: [PATCH] go: add binding for notmuch_message_get_date
In-Reply-To: <m261aijbj5.fsf@att.com>
References: <m2fv9np216.fsf@att.com> <m261aimfcm.fsf@guru.guru-group.fi>
	<87385mmasu.fsf@maritornes.cs.unb.ca> <m261aijbj5.fsf@att.com>
User-Agent: Notmuch/0.19+67~gdbe9924 (http://notmuchmail.org) Emacs/24.4.1
	(x86_64-pc-linux-gnu)
Date: Fri, 06 Mar 2015 08:20:52 +0100
Message-ID: <87wq2ua8x7.fsf@maritornes.cs.unb.ca>
MIME-Version: 1.0
Content-Type: text/plain
Precedence: list

Trevor Jim <tjim@mac.com> writes:


> I'm not sure what the "current troubles" are (happy to hear more), but
> the way I've handled the nil case is in keeping with the way it's
> handled in the other function bindings for go.  Adding an error return
> to all relevant functions seems like a big change that should be handled
> separately, yes?

It seems like at least some functions [1] already use multiple return values
to return a status value.  There is also already a STATUS_NULL_POINTER
defined.  

[1]: from notmuch.go

func NewDatabase(path string) (*Database, Status) {
func OpenDatabase(path string, mode DatabaseMode) (*Database, Status) {
func (self *Database) Close() Status {
func (self *Database) GetDirectory(path string) (*Directory, Status) {
func (self *Database) AddMessage(fname string) (*Message, Status) {
func (self *Database) RemoveMessage(fname string) Status {
func (self *Database) FindMessage(message_id string) (*Message, Status) {
func (self *Message) AddTag(tag string) Status {
func (self *Message) RemoveTag(tag string) Status {
func (self *Message) RemoveAllTags() Status {
func (self *Message) Freeze() Status {
func (self *Message) Thaw() Status {