# HG changeset patch # User Greg Ward # Date 1318556574 14400 # Node ID 7ab05d7524058368bf34fc89d2044461a4723fcd # Parent dd03d3a9f888016b53f024713f0d671cafcdcf88 largefiles: cosmetics, whitespace, code style This is mainly about keeping code under the 80-column limit with as few backslashes as possible. I am deliberately not making any logic or behaviour changes here and have restrained myself to a few "peephole" refactorings. diff -r dd03d3a9f888 -r 7ab05d752405 hgext/largefiles/basestore.py --- a/hgext/largefiles/basestore.py Thu Oct 13 20:45:49 2011 -0400 +++ b/hgext/largefiles/basestore.py Thu Oct 13 21:42:54 2011 -0400 @@ -166,8 +166,8 @@ ui = repo.ui if not remote: - path = getattr(repo, 'lfpullsource', None) or \ - ui.expandpath('default-push', 'default') + path = (getattr(repo, 'lfpullsource', None) or + ui.expandpath('default-push', 'default')) # ui.expandpath() leaves 'default-push' and 'default' alone if # they cannot be expanded: fallback to the empty string, diff -r dd03d3a9f888 -r 7ab05d752405 hgext/largefiles/lfcommands.py --- a/hgext/largefiles/lfcommands.py Thu Oct 13 20:45:49 2011 -0400 +++ b/hgext/largefiles/lfcommands.py Thu Oct 13 21:42:54 2011 -0400 @@ -383,9 +383,9 @@ # if it exists and its hash matches, it might have been locally # modified before updating and the user chose 'local'. in this case, # it will not be in any store, so don't look for it. - if (not os.path.exists(repo.wjoin(lfile)) \ - or expectedhash != lfutil.hashfile(repo.wjoin(lfile))) and \ - not lfutil.findfile(repo, expectedhash): + if ((not os.path.exists(repo.wjoin(lfile)) or + expectedhash != lfutil.hashfile(repo.wjoin(lfile))) and + not lfutil.findfile(repo, expectedhash)): toget.append((lfile, expectedhash)) if toget: @@ -441,9 +441,9 @@ if os.path.exists(absstandin+'.orig'): shutil.copyfile(abslfile, abslfile+'.orig') expecthash = lfutil.readstandin(repo, lfile) - if expecthash != '' and \ - (not os.path.exists(abslfile) or \ - expecthash != lfutil.hashfile(abslfile)): + if (expecthash != '' and + (not os.path.exists(abslfile) or + expecthash != lfutil.hashfile(abslfile))): if not lfutil.copyfromcache(repo, expecthash, lfile): return None # don't try to set the mode or update the dirstate ret = 1 @@ -468,7 +468,6 @@ # -- hg commands declarations ------------------------------------------------ - cmdtable = { 'lfconvert': (lfconvert, [('s', 'size', '', diff -r dd03d3a9f888 -r 7ab05d752405 hgext/largefiles/lfutil.py --- a/hgext/largefiles/lfutil.py Thu Oct 13 20:45:49 2011 -0400 +++ b/hgext/largefiles/lfutil.py Thu Oct 13 21:42:54 2011 -0400 @@ -86,8 +86,8 @@ path = os.path.join(path, hash) else: if os.name == 'nt': - path = os.path.join(os.getenv('LOCALAPPDATA') or \ - os.getenv('APPDATA'), longname, hash) + appdata = os.getenv('LOCALAPPDATA', os.getenv('APPDATA')) + path = os.path.join(appdata, longname, hash) elif os.name == 'posix': path = os.path.join(os.getenv('HOME'), '.' + longname, hash) else: @@ -184,7 +184,8 @@ matcher = getstandinmatcher(repo) # ignore unknown files in working directory - return [splitstandin(f) for f in repo[rev].walk(matcher) \ + return [splitstandin(f) + for f in repo[rev].walk(matcher) if rev is not None or repo.dirstate[f] != '?'] def incache(repo, hash): @@ -394,8 +395,9 @@ def getexecutable(filename): mode = os.stat(filename).st_mode - return (mode & stat.S_IXUSR) and (mode & stat.S_IXGRP) and (mode & \ - stat.S_IXOTH) + return ((mode & stat.S_IXUSR) and + (mode & stat.S_IXGRP) and + (mode & stat.S_IXOTH)) def getmode(executable): if executable: diff -r dd03d3a9f888 -r 7ab05d752405 hgext/largefiles/overrides.py --- a/hgext/largefiles/overrides.py Thu Oct 13 20:45:49 2011 -0400 +++ b/hgext/largefiles/overrides.py Thu Oct 13 21:42:54 2011 -0400 @@ -92,8 +92,9 @@ continue if exact or not exists: - if large or (lfsize and os.path.getsize(repo.wjoin(f)) >= \ - lfsize * 1024 * 1024) or (lfmatcher and lfmatcher(f)): + abovemin = (lfsize and + os.path.getsize(repo.wjoin(f)) >= lfsize * 1024 * 1024) + if large or abovemin or (lfmatcher and lfmatcher(f)): lfnames.append(f) if ui.verbose or not exact: ui.status(_('adding %s as a largefile\n') % m.rel(f)) @@ -117,8 +118,9 @@ else: lfdirstate.add(f) lfdirstate.write() - bad += [lfutil.splitstandin(f) for f in lfutil.repo_add(repo, - standins) if f in m.files()] + bad += [lfutil.splitstandin(f) + for f in lfutil.repo_add(repo, standins) + if f in m.files()] finally: wlock.release() @@ -143,8 +145,9 @@ s = repo.status(match=m, clean=True) finally: repo.lfstatus = False - modified, added, deleted, clean = [[f for f in list if lfutil.standin(f) \ - in manifest] for list in [s[0], s[1], s[3], s[6]]] + modified, added, deleted, clean = [[f for f in list + if lfutil.standin(f) in manifest] + for list in [s[0], s[1], s[3], s[6]]] def warn(files, reason): for f in files: @@ -358,9 +361,10 @@ m._files = [lfutil.standin(f) for f in m._files if lfile(f)] m._fmap = set(m._files) orig_matchfn = m.matchfn - m.matchfn = lambda f: lfutil.isstandin(f) and \ - lfile(lfutil.splitstandin(f)) and \ - orig_matchfn(lfutil.splitstandin(f)) or None + m.matchfn = lambda f: (lfutil.isstandin(f) and + lfile(lfutil.splitstandin(f)) and + orig_matchfn(lfutil.splitstandin(f)) or + None) return m oldmatch = installmatchfn(override_match) listpats = [] @@ -744,8 +748,8 @@ for f in mc: if mc[f] != mp1.get(f, None) or mc[f] != mp2.get(f, None): files.add(f) - toupload = toupload.union(set([f for f in files if lfutil.isstandin(f)\ - and f in ctx])) + toupload = toupload.union( + set([f for f in files if lfutil.isstandin(f) and f in ctx])) return toupload def override_outgoing(orig, ui, repo, dest=None, **opts): diff -r dd03d3a9f888 -r 7ab05d752405 hgext/largefiles/proto.py --- a/hgext/largefiles/proto.py Thu Oct 13 20:45:49 2011 -0400 +++ b/hgext/largefiles/proto.py Thu Oct 13 21:42:54 2011 -0400 @@ -12,9 +12,9 @@ import lfutil -LARGEFILES_REQUIRED_MSG = '\nThis repository uses the largefiles extension.' \ - '\n\nPlease enable it in your Mercurial config ' \ - 'file.\n' +LARGEFILES_REQUIRED_MSG = ('\nThis repository uses the largefiles extension.' + '\n\nPlease enable it in your Mercurial config ' + 'file.\n') def putlfile(repo, proto, sha): '''Put a largefile into a repository's local cache and into the diff -r dd03d3a9f888 -r 7ab05d752405 hgext/largefiles/reposetup.py --- a/hgext/largefiles/reposetup.py Thu Oct 13 20:45:49 2011 -0400 +++ b/hgext/largefiles/reposetup.py Thu Oct 13 21:42:54 2011 -0400 @@ -30,8 +30,8 @@ method = getattr(repo, name) #if not (isinstance(method, types.MethodType) and # method.im_func is repo.__class__.commitctx.im_func): - if isinstance(method, types.FunctionType) and method.func_name == \ - 'wrap': + if (isinstance(method, types.FunctionType) and + method.func_name == 'wrap'): ui.warn(_('largefiles: repo method %r appears to have already been' ' wrapped by another extension: ' 'largefiles may behave incorrectly\n') @@ -193,16 +193,18 @@ lfiles = (modified, added, removed, missing, [], [], clean) result = list(result) # Unknown files - result[4] = [f for f in unknown if repo.dirstate[f] == '?'\ - and not lfutil.isstandin(f)] + result[4] = [f for f in unknown + if (repo.dirstate[f] == '?' and + not lfutil.isstandin(f))] # Ignored files must be ignored by both the dirstate and # lfdirstate result[5] = set(ignored).intersection(set(result[5])) # combine normal files and largefiles - normals = [[fn for fn in filelist if not \ - lfutil.isstandin(fn)] for filelist in result] - result = [sorted(list1 + list2) for (list1, list2) in \ - zip(normals, lfiles)] + normals = [[fn for fn in filelist + if not lfutil.isstandin(fn)] + for filelist in result] + result = [sorted(list1 + list2) + for (list1, list2) in zip(normals, lfiles)] else: def toname(f): if lfutil.isstandin(f): @@ -251,8 +253,8 @@ # Case 1: user calls commit with no specific files or # include/exclude patterns: refresh and commit all files that # are "dirty". - if (match is None) or (not match.anypats() and not \ - match.files()): + if ((match is None) or + (not match.anypats() and not match.files())): # Spend a bit of time here to get a list of files we know # are modified so we can compare only against those. # It can cost a lot of time (several seconds) @@ -345,8 +347,8 @@ fstandin += os.sep # prevalidate matching standin directories - if lfutil.any_(st for st in match._files if \ - st.startswith(fstandin)): + if lfutil.any_(st for st in match._files + if st.startswith(fstandin)): continue actualfiles.append(f) match._files = actualfiles @@ -369,8 +371,8 @@ toupload = set() o = repo.changelog.nodesbetween(o, revs)[0] for n in o: - parents = [p for p in repo.changelog.parents(n) if p != \ - node.nullid] + parents = [p for p in repo.changelog.parents(n) + if p != node.nullid] ctx = repo[n] files = set(ctx.files()) if len(parents) == 2: @@ -388,8 +390,10 @@ None): files.add(f) - toupload = toupload.union(set([ctx[f].data().strip() for f\ - in files if lfutil.isstandin(f) and f in ctx])) + toupload = toupload.union( + set([ctx[f].data().strip() + for f in files + if lfutil.isstandin(f) and f in ctx])) lfcommands.uploadlfiles(ui, self, remote, toupload) return super(lfiles_repo, self).push(remote, force, revs, newbranch)