# HG changeset patch # User Marcin Kuzminski # Date 2013-05-22 03:17:42 # Node ID f5a1314886ec233863a7c3da371cec77b3c60971 # Parent b84c83b651de3bd1de31cd247a0901b6e217428e replace list appends with list literals when possible diff --git a/rhodecode/controllers/feed.py b/rhodecode/controllers/feed.py --- a/rhodecode/controllers/feed.py +++ b/rhodecode/controllers/feed.py @@ -87,9 +87,8 @@ class FeedController(BaseRepoController) return diff_processor, changes def __get_desc(self, cs): - desc_msg = [] - desc_msg.append((_('%s committed on %s') - % (h.person(cs.author), h.fmt_date(cs.date))) + '
') + desc_msg = [(_('%s committed on %s') + % (h.person(cs.author), h.fmt_date(cs.date))) + '
'] #branches, tags, bookmarks if cs.branch: desc_msg.append('branch: %s
' % cs.branch) diff --git a/rhodecode/lib/helpers.py b/rhodecode/lib/helpers.py --- a/rhodecode/lib/helpers.py +++ b/rhodecode/lib/helpers.py @@ -598,11 +598,9 @@ def action_parser(user_log, feed=False, 'ref_name': _name }) revs.append(_rev) - cs_links = [] - cs_links.append(" " + ', '.join( + cs_links = [" " + ', '.join( [lnk(rev, repo_name) for rev in revs[:revs_limit]] - ) - ) + )] _op1, _name1 = _get_op(revs_ids[0]) _op2, _name2 = _get_op(revs_ids[-1]) diff --git a/rhodecode/lib/middleware/simplegit.py b/rhodecode/lib/middleware/simplegit.py --- a/rhodecode/lib/middleware/simplegit.py +++ b/rhodecode/lib/middleware/simplegit.py @@ -55,8 +55,7 @@ class SimpleGitUploadPackHandler(dulserv self.progress("counting objects: %d, done.\n" % len(objects_iter)) dulserver.write_pack_objects(dulserver.ProtocolFile(None, write), objects_iter) - messages = [] - messages.append('thank you for using rhodecode') + messages = ['thank you for using rhodecode'] for msg in messages: self.progress(msg + "\n") diff --git a/rhodecode/lib/vcs/backends/git/config.py b/rhodecode/lib/vcs/backends/git/config.py --- a/rhodecode/lib/vcs/backends/git/config.py +++ b/rhodecode/lib/vcs/backends/git/config.py @@ -318,9 +318,7 @@ class StackedConfig(Config): specified), the users' home directory and the system configuration. """ - paths = [] - paths.append(os.path.expanduser("~/.gitconfig")) - paths.append("/etc/gitconfig") + paths = [os.path.expanduser("~/.gitconfig"), "/etc/gitconfig"] backends = [] for path in paths: try: diff --git a/rhodecode/lib/vcs/backends/git/repository.py b/rhodecode/lib/vcs/backends/git/repository.py --- a/rhodecode/lib/vcs/backends/git/repository.py +++ b/rhodecode/lib/vcs/backends/git/repository.py @@ -635,9 +635,7 @@ class GitRepository(BaseRepository): Tries to pull changes from external location. """ url = self._get_url(url) - cmd = ['pull'] - cmd.append("--ff-only") - cmd.append(url) + cmd = ['pull', "--ff-only", url] cmd = ' '.join(cmd) # If error occurs run_git_command raises RepositoryError already self.run_git_command(cmd)