Hi Gabriel.
Thanks for the answer. I guess I automatically assumed that if a feature
was registered, then it would show up as a default feature in CIL. Again,
my use case for the inliner is very specific, so I'm not sure how many
people would like the inliner to be available by default. Maybe other users
can chime in on this one.
Thanks for investigating the modules -- I can try and work on a patch when
I have some time too.
Bset,
Jon.
On 13 November 2013 05:05, Gabriel Kerneis <gabr...@kerneis.info> wrote:
> 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
