On Fri, Aug 17, 2012 at 6:30 PM, Simon Glass <[email protected]> wrote: > Hi Otavio, > > On Wed, Aug 15, 2012 at 12:44 PM, Otavio Salvador > <[email protected]> wrote: >> In case an address is listed in the To list, those will be skipped on >> Cc list or user might end with a duplicated message. >> >> This fixes the case when a tag points to same address used as series >> destination thus avoiding duplicated sending. >> >> Signed-off-by: Otavio Salvador <[email protected]> >> --- >> tools/patman/series.py | 6 ++++++ >> 1 file changed, 6 insertions(+) >> >> diff --git a/tools/patman/series.py b/tools/patman/series.py >> index eda1e9b..c338ccb 100644 >> --- a/tools/patman/series.py >> +++ b/tools/patman/series.py >> @@ -114,6 +114,12 @@ class Series(dict): >> cc_list += gitutil.BuildEmailList(commit.tags) >> cc_list += gitutil.BuildEmailList(commit.cc_list) >> >> + # Remove items on To list >> + try: >> + map(cc_list.remove, gitutil.BuildEmailList(self.to)) >> + except ValueError: >> + pass >> + > > I'm ok with this as is. But could you do something like: > > if 'to' in self: > map(cc_list.remove, gitutil.BuildEmailList(self.to))
Why? I didn't get why this check would be need. I can do that for sure but I'd like to understand. -- Otavio Salvador O.S. Systems E-mail: [email protected] http://www.ossystems.com.br Mobile: +55 53 9981-7854 http://projetos.ossystems.com.br _______________________________________________ U-Boot mailing list [email protected] http://lists.denx.de/mailman/listinfo/u-boot
