On Thu, Apr 12, 2012 at 05:13:26PM -0000, Gary Lasker wrote:
> Hi mvo! Thanks for the review. You make a very good point about that
> variable name, thanks. It really does not reflect its purpose
> clearly at all. I'm happy with
> self.force_default_search_sort_method, but, hmm, when I see "force"
> it makes me think we are pushing something that shouldn't be
> pushed. Plus, it sounds like a method, and not a flag. How about
> "self.default_search_sort_method_needs_reset"? I'll push a branch
> with that and if it seems fine to you we can merge it.
Sure, good point! I don't mind much what name to use, I used a similar
one in the lp:~mvo/software-center/appview-tweaks branch, but I will
be happy to change it to a different name and/or a different method
name.
On Thu, Apr 12, 2012 at 05:13:26PM -0000, Gary Lasker wrote:
> Hi mvo! Thanks for the review. You make a very good point about that default_ search_ sort_method, but, hmm, when I see "force" search_ sort_method_ needs_reset" ? I'll push a branch
> variable name, thanks. It really does not reflect its purpose
> clearly at all. I'm happy with
> self.force_
> it makes me think we are pushing something that shouldn't be
> pushed. Plus, it sounds like a method, and not a flag. How about
> "self.default_
> with that and if it seems fine to you we can merge it.
Sure, good point! I don't mind much what name to use, I used a similar
one in the lp:~mvo/software-center/appview-tweaks branch, but I will
be happy to change it to a different name and/or a different method
name.
Cheers,
Michael