Colorize attaches properties to the String

[siddarth]

Ruby 1.8.7 (2011-12-28 patchlevel 357) [universal-darwin11.0].

>> require 'colorize'
=> true
>> require 'yaml'
=> true
>> s = 'haha'
=> "haha"
>> puts s.colorize(:background => :red)
haha
=> nil
>> s
=> "haha"
>> YAML.dump(s)
=> "--- !str \nstr: haha\n\"@color\": 9\n\"@mode\": 0\n\"@uncolorized\": !str \n  str: haha\n  \"@color\": 9\n  \"@mode\": 0\n"

[siddarth]

(Got around this by duping the String, but figured I'd throw this here in case other people were having similar issues).

[Wardrop]

Yep, another side-effect of this is it causes an exception to be raised when colorised is called on a frozen string. If you're going to monkey patch something like String, I would never create instance variables if I could at all avoid it.

[fazibear]

In class_parameters branch https://github.com/fazibear/colorize/tree/class_parameters, i moved colorize parameters to class variable. It should work in this case.

[fazibear]

New refactored version don't use any properties.