diff options
| author | Emilio Jesus Gallego Arias | 2018-01-27 02:56:03 +0100 |
|---|---|---|
| committer | Emilio Jesus Gallego Arias | 2018-01-30 19:29:17 +0100 |
| commit | 779c0bd23bbc0bd1d0c1cb358fe9725e7d7ccc74 (patch) | |
| tree | 213309c2c8da155bb4e28e8f668129781e6b5baa /dev | |
| parent | 879ebad4d0b39fda275a72ba44c1f4dfbb9282e5 (diff) | |
[lib] Respect change of options under with/without_option.
The old semantics of `with/without_option` allowed the called function
to modify the value of the option. This is an issue mainly with the
`silently/verbose` combinators, as `Set Silent` can be executed under
one of them and thus the modification will be lost in the updated code
introduced in a554519874c15d0a790082e5f15f3dc2419c6c38
IMHO these kind of semantics are quite messy but we have to preserve
them in order for the `Silent` system to work. In fact, note that in
the previous code, `with_options` was not consistent with
`with_option` [maybe that got me confused?]
Ideally we could restore the saner semantics once we clean up the
`Silent` system [that is, we remove the flag altogether], but that'll
have to wait.
Fixes #6645.
Diffstat (limited to 'dev')
0 files changed, 0 insertions, 0 deletions
