search

lintool / twitter-tools

Twitter Tools
twittertools.cc
218 stars 100 forks source link

Cbor format #48

Open isoboroff opened 9 years ago

isoboroff commented 9 years ago

I need the twittertools crawler to be able to output multiple formats. Well, I need to add support for outputting CBOR.

I'm not proud of this patch, but just can't get up the gumption to design something better... every time I try to type 'CrawlerOutputFormatFactory' my brain seizes up and I go read Twitter. So I'm submitting this pull request mostly in hopes that someone will be affronted enough to write it nicer. But if not please pull this.

isoboroff commented 9 years ago

I've updated this pull request. The current code has been fully tested and generates output that is compatible with what we need for the DD track.

Jimmy, did you want to review before I merge this?