diff options
| author | Chunyan Liu | 2014-06-05 11:21:12 +0200 |
|---|---|---|
| committer | Stefan Hajnoczi | 2014-06-16 11:23:21 +0200 |
| commit | 98d896d978eef0fd6ff8d93a1ccea34e00b97f8c (patch) | |
| tree | 1c2a049cb1aebe3fe24aec7494dde091351aac6d /include | |
| parent | cleanup QEMUOptionParameter (diff) | |
| download | qemu-98d896d978eef0fd6ff8d93a1ccea34e00b97f8c.tar.gz qemu-98d896d978eef0fd6ff8d93a1ccea34e00b97f8c.tar.xz qemu-98d896d978eef0fd6ff8d93a1ccea34e00b97f8c.zip | |
QemuOpts: cleanup tmp 'allocated' member from QemuOptsList
Now only qemu_opts_append uses 'allocated' to indicate free memory.
For this function only, we can also let result list's (const char *)
members point to input list's members, only if the input list has
longer lifetime than result list. In current code, that is true.
So, we can remove the 'allocated' member from QemuOptsList definition
to keep code clean.
Signed-off-by: Chunyan Liu <cyliu@suse.com>
Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>
Diffstat (limited to 'include')
| -rw-r--r-- | include/qemu/option.h | 6 |
1 files changed, 0 insertions, 6 deletions
diff --git a/include/qemu/option.h b/include/qemu/option.h index 921eccdb38..59bea759a2 100644 --- a/include/qemu/option.h +++ b/include/qemu/option.h @@ -63,12 +63,6 @@ typedef struct QemuOptDesc { } QemuOptDesc; struct QemuOptsList { - /* FIXME: Temp used for QEMUOptionParamter->QemuOpts conversion to - * indicate the need to free memory. Will remove after all drivers - * switch to QemuOpts. - */ - bool allocated; - const char *name; const char *implied_opt_name; bool merge_lists; /* Merge multiple uses of option into a single list? */ |