It's deprecated. It calls `self.class.protocol_separator`, which has the deprecation message.
Should I add a comment to make that more clear?
You aren't the first person to be confused about this.

--
Kasper

> Den 03/09/2013 kl. 10.05 skrev Robin Dupret <notifications@github.com>:
> 
> In actionview/lib/action_view/helpers/sanitize_helper/sanitizers.rb:
> 
> > +        remove_xpaths(loofah_fragment, XPATHS_TO_REMOVE)
> > +        loofah_fragment.scrub!(:strip)
> > +      end
> > +      loofah_fragment.to_s
> > +    end
> > +
> > +    def sanitize_css(style_string)
> > +      Loofah::HTML5::Scrub.scrub_css style_string
> > +    end
> > +
> > +    def protocol_separator
> > +      self.class.protocol_separator
> > +    end
> > +
> > +    def protocol_separator=(value)
> > +      self.class.protocol_separator
> I'm not sure to understand here. Do you mean that we can't change the value of the protocol separator ? If it's the case, then we should better put a deprecation warning too, no ? Sorry if I'm misunderstanding.
> 
> —
> Reply to this email directly or view it on GitHub.