Rev 5430: Simplify set_default_merge_tool. in file:///home/vila/src/bzr/reviews/mergetools/
Vincent Ladeuil
v.ladeuil+lp at free.fr
Mon Dec 6 14:37:41 GMT 2010
At file:///home/vila/src/bzr/reviews/mergetools/
------------------------------------------------------------
revno: 5430
revision-id: v.ladeuil+lp at free.fr-20101206143741-eqbgbmmgn6ya8oor
parent: v.ladeuil+lp at free.fr-20101206141515-s6oa96cj01ley5s3
committer: Vincent Ladeuil <v.ladeuil+lp at free.fr>
branch nick: mergetools
timestamp: Mon 2010-12-06 15:37:41 +0100
message:
Simplify set_default_merge_tool.
-------------- next part --------------
=== modified file 'bzrlib/config.py'
--- a/bzrlib/config.py 2010-12-06 13:33:47 +0000
+++ b/bzrlib/config.py 2010-12-06 14:37:41 +0000
@@ -399,15 +399,10 @@
return tool
def set_default_merge_tool(self, name):
- if name is None:
- self.remove_user_option('bzr.default_mergetool')
- else:
- if isinstance(name, mergetools.MergeTool):
- name = name.get_name()
- if self._find_merge_tool(name) is None:
- raise errors.BzrError('invalid merge tool name: %r' % name)
- trace.mutter('setting default merge tool: %s', name)
- self.set_user_option('bzr.default_mergetool', name)
+ if self._find_merge_tool(name) is None:
+ raise errors.BzrError('invalid merge tool name: %r' % name)
+ trace.mutter('setting default merge tool: %s', name)
+ self.set_user_option('bzr.default_mergetool', name)
class IniBasedConfig(Config):
=== modified file 'bzrlib/tests/test_config.py'
--- a/bzrlib/tests/test_config.py 2010-12-06 10:33:58 +0000
+++ b/bzrlib/tests/test_config.py 2010-12-06 14:37:41 +0000
@@ -1082,11 +1082,6 @@
self.assertEqual('winmergeu',
conf.get_user_option('bzr.default_mergetool'))
- def test_set_default_merge_tool_none(self):
- conf = config.GlobalConfig.from_string(sample_config_text)
- conf.set_default_merge_tool(None)
- self.assertEqual(None, conf.get_user_option('bzr.default_mergetool'))
-
def test_set_invalid_default_merge_tool(self):
conf = config.GlobalConfig()
tools = [
More information about the bazaar-commits
mailing list