# HG changeset patch # User Christian Ebert # Date 1286669236 -3600 # Node ID 448a71a24a84ae270f33b19c8c26878138c44cc0 # Parent 53a9228f13dee8e4eee471beff9e4d538d337cbd enforce subn method via boolean switch There are only 2 patterns to choose, and so far only 1 case where kwtemplater.re_kw.subn is applied on data read from the working directory: when recording added files. With this change the code reflects more closely the boolean character of the switch and underlines the special case. diff -r 53a9228f13de -r 448a71a24a84 hgkw/keyword.py --- a/hgkw/keyword.py Sun Oct 10 01:07:16 2010 +0100 +++ b/hgkw/keyword.py Sun Oct 10 01:07:16 2010 +0100 @@ -194,7 +194,7 @@ expansion are not symbolic links.''' return [f for f in cand if self.match(f) and not 'l' in ctx.flags(f)] - def overwrite(self, ctx, candidates, lookup, expand, recsubn=None): + def overwrite(self, ctx, candidates, lookup, expand, rekw=False): '''Overwrites selected files expanding/shrinking keywords.''' if self.restrict or lookup: # exclude kw_copy candidates = self.iskwfile(candidates, ctx) @@ -204,8 +204,7 @@ if self.restrict or expand and lookup: mf = ctx.manifest() fctx = ctx - subn = (self.restrict and self.re_kw.subn or - recsubn or self.re_kwexp.subn) + subn = (self.restrict or rekw) and self.re_kw.subn or self.re_kwexp.subn msg = (expand and _('overwriting %s expanding keywords\n') or _('overwriting %s shrinking keywords\n')) for f in candidates: @@ -578,8 +577,8 @@ modified = [f for f in modified if f in changed] added = [f for f in added if f in changed] kwt.restrict = False - kwt.overwrite(recctx, modified, False, True, kwt.re_kwexp.subn) - kwt.overwrite(recctx, added, False, True, kwt.re_kw.subn) + kwt.overwrite(recctx, modified, False, True) + kwt.overwrite(recctx, added, False, True, True) kwt.restrict = True return ret finally: