[whups] Re: library conventions and state of play

[Chuck Hagenbuch]

Quoting Anil Madhavapeddy <anil@recoil.org>:

> I decided that the human-readable diff was just a html rendition
> of the unified diff, and so shouldn't really have its own class
> (since it is presentation layer).

That's not necessarily a reason. Model-View-Controller is all about 
_seperating_ the presentation layer, but the presentation layer is still 
something that you design - there's some logic there, especially in 
representing diffs - so I think there's a good reason to put different views 
into their own CVSLib_diffView_* classes.

-chuck

--
Charles Hagenbuch, <chuck@horde.org>
"If you can't stand the heat, get out of the chicken!" - Baby Blues