Feature/generate to writer #19

Closed
wants to merge 2 commits into
from

2 participants

@KirinDave

Turns out I was generating some fairly large json strings and I wanted to be able to use other writers besides a StringWriter. This is the smallest change one can possibly make to support that while eliminating unnecessary duplication.

@lancepantz
Collaborator

I modernized your typehint (OCD), wasn't sure whether to re-write your commit, or add another. I re-wrote it, sorry if that's dick. You'll have to update your tracking branch.

Released to clojars.

Oh, it's fine. I don't mind either way. Thanks for the fast turnaround time.

You can always ask me to redo the commit as well. For some reason I always reflexively use the old typehints when I am working fast. My apologies for that.

Collaborator

np, Now hook me up with a banksimple invite :)

I do not work there anymore; haven't for a year. This is part of ongoing efforts at CrowdFlower. Sorry!

Collaborator

ah cool, take care dude!

@lancepantz lancepantz closed this Nov 20, 2012
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment