rspec/rspec-core — pull request #659 by moredip (2012-08-04 20:23:32+00:00)
Title: Pr/html formatter cleanup

I wanted to do some work on extending the HtmlFormatter, but I had a hard time reading through the existing implementation. It felt like the logic was hidden in between all the `@output.puts` calls.

This patch attempts to fix that by extracting an HtmlPrinter which knows what the HTML content should look like but contains pretty much no logic. The HtmlFormatter holds the logic and delegates to the printer for actual HTML generation. Seems like a nice separation of concerns to me.

Assuming this is considered a good direction I'd like to continue with additional cleanup if that's OK.å