diff options
author | Aleksey Midenkov <midenok@gmail.com> | 2022-07-18 23:16:17 +0300 |
---|---|---|
committer | Aleksey Midenkov <midenok@gmail.com> | 2022-07-18 23:16:17 +0300 |
commit | 220fb6797b7447d500f1103d5ce556255939c762 (patch) | |
tree | 648fe1fd5b845dd99dea9fc340654bb9039857cc /randgen | |
parent | ce7820eb838c8413fb0fe1d3812c43824b508eab (diff) | |
download | mariadb-git-220fb6797b7447d500f1103d5ce556255939c762.tar.gz |
MDEV-28931 Debugger.pm readability fix
setup_boot_args(), setup_client_args(), setup_args() traversing
datastructures on each invocation. Even if performance is not
important to perl script (though it definitely saves some CO2), this
nonetheless provokes some code-reading questions. Reading and
debugging such code is not convenient.
The better way is to prepare all the data in advance in an easily
readable form as well as do the validation step before any further
processing.
Use mtr_report() instead of die() like the other code does.
TODO: do_args() does even more data processing magic. Prepare that
data according the above strategy in advance in pre_setup() if possible.
Diffstat (limited to 'randgen')
0 files changed, 0 insertions, 0 deletions