comparison COPYING @ 30480:b0a8337ba9af

ui: add configoverride context manager I feel like this idea might've been discussed before, so please feel free to point me to the right mailing list entry to read about why it should not be done. We have a common pattern of the following code: backup = ui.backupconfig(section, name) try: ui.setconfig(section, name, temporaryvalue, source) do_something() finally: ui.restoreconfig(backup) IMO, this looks better: with ui.configoverride({(section, name): temporaryvalue}, source): do_something() Especially this becomes more convenient when one has to backup multiple config values before doing something. In such case, adding a new value to backup requires codemod in three places.
author Kostia Balytskyi <ikostia@fb.com>
date Mon, 21 Nov 2016 16:22:26 -0800
parents 8c8b55733cbd
children
comparison
equal deleted inserted replaced
30479:798bcb1274dd 30480:b0a8337ba9af