<nomunge> is the wrong way around, false causes no munging

davidB / yuicompressor-maven-plugin

maven's plugin to compress (Minify / Ofuscate / Aggregate) Javascript files and CSS files using YUI Compressor

http://davidb.github.io/yuicompressor-maven-plugin/

GNU Lesser General Public License v2.1

122 stars 48 forks source link

<nomunge> is the wrong way around, false causes no munging #50

Closed spudbean closed 12 years ago

spudbean commented 12 years ago

Currently in 1.3.0: When is specified, "false" causes local variables to be preserved (i.e. it is not munging) and "true" causes local variables to be optimised (i.e. it is munging*).

Desired:

nomunge=false should mean munge=true should mean "please munge my local variables"
nomunge=true should mean munge=false should mean "please leave my local variables alone"

It may be problematic to change the meaning of this parameter in a backward-compatible way.

spudbean commented 12 years ago

User error -- could not reproduce.