On Fri, Mar 22, 2013 at 12:26:50AM -0700, Alexis Metaireau wrote:
> > @@ -169,6 +172,9 @@ def _update_content(self, content, siteurl):
> >          :param siteurl: siteurl which is locally generated by the writer in
> >                          case of RELATIVE_URLS.
> >          """
> > +        if not content:
> > +            return content
> 
> maybe we should make this consistent and be sure we return a string?
> "return ''"

The only way `content` should be `None` is for `Content` instances
where you don't care about the content (i.e. `Static`, since you just
copy the files without peeking inside).  I don't see any reason to
return a string in that case.

If you're worried about the other `Content` subclasses, if content
starts out as the empty string, it's going to end up as the empty
string anyway, so cutting out early has no effect.