html_safe being called when it shouldn't

[ghost]

A snippet from textilize method, note that html_safe is being called on the return value of the if block. Which means it will be called even if "" is returned, which is unnecessary.

  if text.blank?
    ""
  else
    textilized = RedCloth.new(text, options)
    textilized.to_html
  end.html_safe

[dtrasbo]

That is the desired behavior. If we don't call html_safe on the blank string it will be sanitized, which is unnecessary because the string obviously is safe.