Remove QEMUFile abuse
Get series with b4
b4 am [email protected]Versions newest first · linked by author and title
- v2 Latest3 patches 1 replyIn review
- v13 patches 6 repliesSuperseded
Patches in v2 No applies recorded yet
| # | Subject and files | Size | Replies | Applied |
|---|---|---|---|---|
| 1/3 | wavaudio: Use stdio instead of QEMUFileaudio/ | +24 | 0 replies | |
| 2/3 | wavcapture: Use stdio instead of QEMUFileaudio/ | +34 | 1 reply | |
| 3/3 | ds1225y: Use stdio instead of QEMUFilehw/ | +16 | 0 replies |
Reviews trailers per patch
No Reviewed-by, Acked-by or Tested-by trailers for v2 yet.
Cover letter [Qemu-devel] [PATCH v2 0/3] Remove QEMUFile abuse
Hi
v2: Make malc happy release O;-)
- learn how to use fwrite/fread correctly O:-)
- add errno to all error messages
- make checkpatch happy. Except for
WARNING: space prohibited between function name and open parenthesis '('
Please apply.
v1:
QEMUFile is intended to be used only for migration. Change the other three users to use FILE * operations directly. gcc on Fedora 15 complains about fread/write not checking its return value, so I added checks. But in several places only print an error message (there is no error handly that I can hook into). Notice that this is not worse than it is today.
Later, Juan.
Juan Quintela (3):
wavaudio: Use stdio instead of QEMUFile
wavcapture: Use stdio instead of QEMUFile
ds1225y: Use stdio instead of QEMUFile
audio/wavaudio.c | 34 ++++++++++++++++++++++++---------- audio/wavcapture.c | 48 ++++++++++++++++++++++++++++++++++-------------- hw/ds1225y.c | 28 ++++++++++++++++------------ 3 files changed, 74 insertions(+), 36 deletions(-)
Signature · 1 line
1.7.6.2