diff options
| author | Daniel Henrique Barboza <danielhb413@gmail.com> | 2018-06-21 07:21:51 -0300 |
|---|---|---|
| committer | Michael Roth <mdroth@linux.vnet.ibm.com> | 2018-07-03 15:20:51 -0500 |
| commit | 8b020b5eb708091e723518907184e609350f6d41 (patch) | |
| tree | 26cd953a71577bdaefb6ff85b3a7f0cd1d7e2973 /scripts/qapi/events.py | |
| parent | 246d76eba1944d7e59affb288ec27d7fcfb5d256 (diff) | |
| download | focaccia-qemu-8b020b5eb708091e723518907184e609350f6d41.tar.gz focaccia-qemu-8b020b5eb708091e723518907184e609350f6d41.zip | |
qga: removing switch statements, adding run_process_child
This is a cleanup of the resulting code after detaching pmutils and Linux sys state file logic: - remove the SUSPEND_MODE_* macros and use an enumeration instead. At the same time, drop the switch statements at the start of each function and use the enumeration index to get the right binary/argument; - create a new function called run_process_child(). This function uses g_spawn_sync() to execute a shell command, returning the exit code. This is a common operation in the pmutils functions and will be used in the systemd implementation as well, so this function will avoid code repetition. There are more places inside commands-posix.c where this new run_process_child function can also be used, but one step at a time. Signed-off-by: Daniel Henrique Barboza <danielhb413@gmail.com> *check/propagate local_err before setting errp directly Signed-off-by: Michael Roth <mdroth@linux.vnet.ibm.com>
Diffstat (limited to 'scripts/qapi/events.py')
0 files changed, 0 insertions, 0 deletions