On Sat, Mar 23, 2013 at 06:42:35PM -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
> 
> that's for consistency: if we expect this to return a string, then
> it should probably return a string (but that's not an important
> thing anyway)

How about bumping this cutout up the stack into `.get_content()`:

    @memoized
    def get_content(self, siteurl):

        if self._content and hasattr(self, '_get_content'):
            content = self._get_content()
        else:
            content = self._content
        return self._update_content(content, siteurl)

although then the inconsistency is in get_content() :p.  At some
point, content is going to be either `None` or a string.  We currently
support both in `._content`, but not in `._update_content`.  I'm fine
with fixing this inconsistency at almost any level, but I don't think
we should be using the empty string (instead of `None`) for
`Static._content`.