Am 17.03.2011 16:49, schrieb Michael Tokarev: > 17.03.2011 18:04, Kevin Wolf wrote: >> Am 17.03.2011 08:58, schrieb Michael Tokarev: > [] >>> --- a/vl.c >>> +++ b/vl.c >>> @@ -2098,7 +2098,8 @@ int main(int argc, char **argv, char **envp) >>> HD_OPTS); >>> break; >>> case QEMU_OPTION_drive: >>> - drive_def(optarg); >>> + if (drive_def(optarg) == NULL) >>> + exit(1); >> >> Coding style requires braces here. > > I'll just stick it into debian package. I'm not going > to change all the other braces like this around that place.
I'm only asking to add one pair of braces in the line that you touch, not to change everything in qemu. Shouldn't be that hard... Kevin
