lltyk/tapestry-wro4j — comment on commit b2593197c97908126c8f6c41ab86e6996bea752b
File: src/main/java/com/github/lltyk/wro4j/services/CoffeeScriptJsTransformer.java line 44
By lltyk:

Well at first it was just not feeling like creating a Resource, which
probably isn't a very good reason. However GoogleClosureCompressorProcessor
depends on your Context/configuration which I do not want to use. Tapestry
addons avoid making people modify their web.xml, and I also want the
tapestry flags to control whether compression is enabled or not, as opposed
to your filter doing it. GoogleClosureCompressorProcessor referencing
Context made me not want to use the *Processor classes in general.

On the other hand I suppose I could create a WroConfiguration and set
Context myself, before using a Processor. Could open the way for
configuring some of the options in there.

Question, I want to pass the reservedNames parameter to UglifiyJS. I
suppose I need to add that as an option in the UglifyJS class side by side
with the uglify/beautify option?

On Thu, Dec 22, 2011 at 3:31 AM, Alex Objelean <
reply@reply.github.com
> wrote:

> First of all, great to see that wro4j is used for tapestry :).
>
> Is there a reason why you prefer using CoffeeScript class instead of
> CoffeeScriptProcessor which hides the implementation details like dealing
> with ObjectPoolHelper? This way, you can benefit of any performance
> improvements there will be related to the way processor is implemented,
> instead of updating the implementation yourself.
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/lltyk/tapestry-wro4j/commit/b2593197c97908126c8f6c41ab86e6996bea752b#commitcomment-815052
>