[Sugar-devel] [PATCH] fix journal scan of external media, dev.laptop.org #10140

James Cameron quozl at laptop.org
Wed Dec 1 19:13:59 EST 2010


On Tue, Nov 30, 2010 at 03:32:57PM +0100, Sascha Silbe wrote:
> Excerpts from James Cameron's message of Tue Nov 30 01:14:14 +0100 2010:
> 
> [src/jarabe/journal/model.py]
> 
> > +        if self._date_start is not None and self.st_mtime < self._date_start:
> > +            return
> > +
> > +        if self._date_end is not None and self.st_mtime > self._date_end:
> > +            return
> 
> Both occurrences of self.st_mtime need to read stat.st_mtime instead
> (found by pylint).

Sigh.  You suggested splitting these tests, expanding the scope of my
patch.  I now regret accepting the suggestion.  I think I should stick
to the changes I want to make, not the changes other people want me to
make.  When enthusiasm for a change is low, the defect rate is high.

-- 
James Cameron
http://quozl.linux.org.au/


More information about the Sugar-devel mailing list