Hi Raynos, I was thinking about your confusing and bloated comment. I was wondering if you were looking at the right files? A couple of things:
1) The Javascript code is compiled from Coffeescript so it is going to look a little strange. What I did with an underscore extension library I wrote (https://github.com/kmalakoff/underscore-awesomer) was first write it in Coffeescript and then when it was stabilized, I hand- optimized the Javascript implementation. I'll do the same with this library if it takes off. -> The file you or anyone else should review: src/mixin_core.coffee 2) If people want to look at the Javascript, take a look at the core library version ./mixin_core.js not the full version that comes bundled with all of the mixins I released (not ./mixin.js unless you want to!) -> Or if you prefer Javascript please review: ./mixin_core.js Any more feedback would be much appreciated! Cheers, Kevin On Oct 11, 12:16 pm, Raynos <[email protected]> wrote: > I've tried looking at the source code. It looks confusing and bloated and it > appears like your writing an OO system on top of javascript. > > It simply looks like overkill > > A mixin shouldn't be any more then `Object.extend(o, mixin)` -- To view archived discussions from the original JSMentors Mailman list: http://www.mail-archive.com/[email protected]/ To search via a non-Google archive, visit here: http://www.mail-archive.com/[email protected]/ To unsubscribe from this group, send email to [email protected]
