Hi Chris, very cool! Thanks for the contribution. I've looked through
it and it looks great, and I verified the tests.
Joe, I assume you're OK with me adding this to the mimeparse collection?
thanks,
--steve
I've added and checked in the files, added a JavaScript example to the
wiki, and added a download file for the JavaScript code to the front
page.
Chris, I did make a change to the code: there was a call to
console.log in bestMatch, and that didn't work in Rhino (which I used
to make the wiki example), so I removed it. I hope that's OK with you.
--steve
Thanks Joe. I'd be honored (and I have some cleanup to do).
On Fri, Dec 19, 2008 at 8:06 AM, Steve Vinoski <vin...@gmail.com> wrote:
> I've added and checked in the files, added a JavaScript example to the
> wiki, and added a download file for the JavaScript code to the front
> page.
Thanks. The only gotcha I see is that json2.js is missing from the
package, which is fine for the sake of mimeparse.js, but leaves the
tests broken here:
http://mimeparse.googlecode.com/svn/trunk/mimeparse-js-test.html
This is something I can fix when I'm added to the project. (assuming
that there's no objection to including json2.js in the repository.
It's public domain, so the only downside is the clutter of a file
needed only for the test runner. Actually, the best thing may be to
paste it inline into the test runner html, so no one gets confused and
thinks that its required to run mimeparse.js)
>
> Chris, I did make a change to the code: there was a call to
> console.log in bestMatch, and that didn't work in Rhino (which I used
> to make the wiki example), so I removed it. I hope that's OK with you.
No problem, Steve -- I have a bad habit of leaving those console.log
statements in. There are a couple others in the failing assertion
path, which I'll gladly clean up as well.
Chris
--
Chris Anderson
http://jchris.mfdz.com
Thanks
Thanks. I've committed the debug-statement free version, and the test
cases now work fine at
http://mimeparse.googlecode.com/svn/trunk/mimeparse-js-test.html
I love testing in the browser!
Chris