Hi Jonathan, On Tue, Nov 12, 2013 at 11:38:08PM -0800, Jonathan Kotker wrote: > This is more a question out of curiosity, but is there a reason that the > inliner, available in src/ext, is not included in the default features of > CIL? I use it as part of GameTime, and currently, when a user installs > GameTime, the installation procedure uses a patch file to add a line to the > bottom of the inliner source code that registers it as a feature.
There are two questions here: - why is the inliner not part of CIL default features? Because that's how it was in the past, and I want to preserve backward-compatibility (not removing default features) and usability (not adding too many features by default). Feel free to argue about that part, I'm prone to change my mind. - why is doesn't the inliner module call Feature.register? This is a bug. I just checked it looks like the following features are affected: blockinggraph, ccl, cqualann, inliner and llvm. Feel free to send a patch (or I'll fix it myself when I have some more time). Thanks for the report, -- Gabriel ------------------------------------------------------------------------------ DreamFactory - Open Source REST & JSON Services for HTML5 & Native Apps OAuth, Users, Roles, SQL, NoSQL, BLOB Storage and External API Access Free app hosting. Or install the open source package on any LAMP server. Sign up and see examples for AngularJS, jQuery, Sencha Touch and Native! http://pubads.g.doubleclick.net/gampad/clk?id=63469471&iu=/4140/ostg.clktrk _______________________________________________ CIL-users mailing list CIL-users@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/cil-users
