JSMin_PHP Library Port

[eluser]tonydewan[/eluser]
I'm not sure why it was written that way, other than for the sake of brevity. You'd have to ask Ryan Grove (or perhaps Doug Crockford.) That is the way the class is written in the original JSMin PHP port. In porting it to CI, I only added the log_message function call and set a default value for $js in the constructor.

Your solution should work, assuming you match variable names (changing $js to $input):

Code:

public static function minify($input) {

    $this->input       = str_replace("\r\n", "\n", $input);
    $this->inputLength = strlen($this->input);
    
    $return = $this->min();

    // Housekeeping
    $this->input       = null;
    $this->inputLength = null;

    return $return;

  }

I still prefer the way it was originally written, if only because it's cleaner. Thanks for the comment!