search

couchbaselabs / cbfs

Distributed Blobstore using Couchbase Server
http://dustin.github.com/2012/09/27/cbfs.html
Apache License 2.0
188 stars 40 forks source link

#138 - only call defer f.Close() when needed #141

Closed tleyden closed 9 years ago

tleyden commented 9 years ago

@mschoch I'm gonna just go ahead and merge this since this feels pretty non-controversial, but take a look when you get a chance.

mschoch commented 9 years ago

Yeah go ahead, I'd like to get out of the cbfs maintenance role. On Dec 20, 2014 3:29 PM, "Traun Leyden" notifications@github.com wrote:

@mschoch https://github.com/mschoch I'm gonna just go ahead and merge this since this feels pretty non-controversial, but take a look when you get a chance.

— Reply to this email directly or view it on GitHub https://github.com/couchbaselabs/cbfs/pull/141#issuecomment-67748500.

tleyden commented 9 years ago

Ok, thanks for letting me know. I will hit up @adamcfraser and @ajres for code reviews going forward.