[Qemu-devel] [PATCH v3 0/4] ui/cocoa.m: Fix compiler and runtime warnings on 10.8
These patches fix various compiler and runtime warnings QEMU provokes on MacOSX 10.8. The first two fix a leak and some deprecated functions which cause warnings in the system log when QEMU runs. The second two avoid some functions which cause compile time warnings about use of functions deprecated in 10.6. I believe from my reading of the documentation that these changes should still work OK on 10.3, but I don't have any way of testing that. Andreas: I don't write much ObjC so I don't really have a feel for what is good coding style. I mostly tried to follow the existing code (eg "//" style comments) and QEMU's C style (eg 80 cols max). I'm happy to change any of the indentation or whatever to suit your preferences. With this patch set the only remaining compile time warnings from clang on 10.8 are about deprecated methods in audio/coreaudio.c. Changes v2->v3: * rebased again Changes v1->v2: * rebased and fixed up following recent console changes Peter Maydell (4): ui/cocoa.m: Fix leaks of NSScreen and NSConcreteMapTable ui/cocoa.m: Avoid deprecated CPS* functions ui/cocoa.m: Avoid deprecated NSOpenPanel filename method ui/cocoa.m: Avoid deprecated NSOpenPanel beginSheetForDirectory ui/cocoa.m | 52 1 file changed, 32 insertions(+), 20 deletions(-) -- 1.7.11.4
Re: [Qemu-devel] [PATCH v3 0/4] ui/cocoa.m: Fix compiler and runtime warnings on 10.8
Ping again! This patchset has now been on the list for eleven weeks without any review. thanks -- PMM On 2 May 2013 11:29, Peter Maydell wrote: > Ping! > > -- PMM > > On 22 April 2013 21:29, Peter Maydell wrote: >> These patches fix various compiler and runtime warnings QEMU provokes >> on MacOSX 10.8. The first two fix a leak and some deprecated functions >> which cause warnings in the system log when QEMU runs. The second >> two avoid some functions which cause compile time warnings about >> use of functions deprecated in 10.6. >> >> I believe from my reading of the documentation that these changes >> should still work OK on 10.3, but I don't have any way of testing that. >> >> Andreas: I don't write much ObjC so I don't really have a feel for >> what is good coding style. I mostly tried to follow the existing >> code (eg "//" style comments) and QEMU's C style (eg 80 cols max). >> I'm happy to change any of the indentation or whatever to suit your >> preferences. >> >> With this patch set the only remaining compile time warnings from clang >> on 10.8 are about deprecated methods in audio/coreaudio.c. >> >> Changes v2->v3: >> * rebased again >> Changes v1->v2: >> * rebased and fixed up following recent console changes >> >> Peter Maydell (4): >> ui/cocoa.m: Fix leaks of NSScreen and NSConcreteMapTable >> ui/cocoa.m: Avoid deprecated CPS* functions >> ui/cocoa.m: Avoid deprecated NSOpenPanel filename method >> ui/cocoa.m: Avoid deprecated NSOpenPanel beginSheetForDirectory >> >> ui/cocoa.m | 52 >> 1 file changed, 32 insertions(+), 20 deletions(-) >> >> -- >> 1.7.11.4 >> >>
Re: [Qemu-devel] [PATCH v3 0/4] ui/cocoa.m: Fix compiler and runtime warnings on 10.8
Peter Maydell writes: > Ping again! This patchset has now been on the list for eleven > weeks without any review. Objective C is a foreign language. Andreas is the Cocoa maintainer so I've been waiting for him. Andreas, if you don't have time to look at these patches, I can apply them directly. Regards, Anthony Liguori > > thanks > -- PMM > > On 2 May 2013 11:29, Peter Maydell wrote: >> Ping! >> >> -- PMM >> >> On 22 April 2013 21:29, Peter Maydell wrote: >>> These patches fix various compiler and runtime warnings QEMU provokes >>> on MacOSX 10.8. The first two fix a leak and some deprecated functions >>> which cause warnings in the system log when QEMU runs. The second >>> two avoid some functions which cause compile time warnings about >>> use of functions deprecated in 10.6. >>> >>> I believe from my reading of the documentation that these changes >>> should still work OK on 10.3, but I don't have any way of testing that. >>> >>> Andreas: I don't write much ObjC so I don't really have a feel for >>> what is good coding style. I mostly tried to follow the existing >>> code (eg "//" style comments) and QEMU's C style (eg 80 cols max). >>> I'm happy to change any of the indentation or whatever to suit your >>> preferences. >>> >>> With this patch set the only remaining compile time warnings from clang >>> on 10.8 are about deprecated methods in audio/coreaudio.c. >>> >>> Changes v2->v3: >>> * rebased again >>> Changes v1->v2: >>> * rebased and fixed up following recent console changes >>> >>> Peter Maydell (4): >>> ui/cocoa.m: Fix leaks of NSScreen and NSConcreteMapTable >>> ui/cocoa.m: Avoid deprecated CPS* functions >>> ui/cocoa.m: Avoid deprecated NSOpenPanel filename method >>> ui/cocoa.m: Avoid deprecated NSOpenPanel beginSheetForDirectory >>> >>> ui/cocoa.m | 52 >>> 1 file changed, 32 insertions(+), 20 deletions(-) >>> >>> -- >>> 1.7.11.4 >>> >>>
Re: [Qemu-devel] [PATCH v3 0/4] ui/cocoa.m: Fix compiler and runtime warnings on 10.8
Am 28.05.2013 19:01, schrieb Anthony Liguori: > Peter Maydell writes: > >> Ping again! This patchset has now been on the list for eleven >> weeks without any review. > > Objective C is a foreign language. For that reason I've been applying our C Coding Style for indentation and spacing, but the code was full of // comments that I did not bother to change. > Andreas is the Cocoa maintainer so > I've been waiting for him. Andreas, if you don't have time to look at > these patches, I can apply them directly. I've looked at the patches today seeing the ping and they look fine - I wasn't properly CC'ed before (get_maintainer.pl would've spared the typo!) and so they were simply not in my mailbox. I'll try to test on v10.5 tonight. Peter and me had already agreed upfront that there's no point in maintaining < v10.3 code any more. BTW if Peter or someone wants to take over Cocoa (co-)maintenance that would be fine with me. Regards, Andreas > > Regards, > > Anthony Liguori > >> >> thanks >> -- PMM >> >> On 2 May 2013 11:29, Peter Maydell wrote: >>> Ping! >>> >>> -- PMM >>> >>> On 22 April 2013 21:29, Peter Maydell wrote: These patches fix various compiler and runtime warnings QEMU provokes on MacOSX 10.8. The first two fix a leak and some deprecated functions which cause warnings in the system log when QEMU runs. The second two avoid some functions which cause compile time warnings about use of functions deprecated in 10.6. I believe from my reading of the documentation that these changes should still work OK on 10.3, but I don't have any way of testing that. Andreas: I don't write much ObjC so I don't really have a feel for what is good coding style. I mostly tried to follow the existing code (eg "//" style comments) and QEMU's C style (eg 80 cols max). I'm happy to change any of the indentation or whatever to suit your preferences. With this patch set the only remaining compile time warnings from clang on 10.8 are about deprecated methods in audio/coreaudio.c. Changes v2->v3: * rebased again Changes v1->v2: * rebased and fixed up following recent console changes Peter Maydell (4): ui/cocoa.m: Fix leaks of NSScreen and NSConcreteMapTable ui/cocoa.m: Avoid deprecated CPS* functions ui/cocoa.m: Avoid deprecated NSOpenPanel filename method ui/cocoa.m: Avoid deprecated NSOpenPanel beginSheetForDirectory ui/cocoa.m | 52 1 file changed, 32 insertions(+), 20 deletions(-) -- 1.7.11.4 > >
Re: [Qemu-devel] [PATCH v3 0/4] ui/cocoa.m: Fix compiler and runtime warnings on 10.8
On 28 May 2013 18:22, Andreas Färber wrote: > I've looked at the patches today seeing the ping and they look fine - I > wasn't properly CC'ed before (get_maintainer.pl would've spared the > typo!) and so they were simply not in my mailbox. Augh, sorry -- no idea where that other address came from. (it doesn't bounce, or I'd have noticed before; I wonder what poor unfortunate has been being sent qemu patches...) > I'll try to test on v10.5 tonight. Peter and me had already agreed > upfront that there's no point in maintaining < v10.3 code any more. > > BTW if Peter or someone wants to take over Cocoa (co-)maintenance that > would be fine with me. I don't object to being listed as a co-maintainer if you think it would be helpful, though the only MacOS system I have is this 10.8 one. thanks -- PMM
Re: [Qemu-devel] [PATCH v3 0/4] ui/cocoa.m: Fix compiler and runtime warnings on 10.8
Am 22.04.2013 22:29, schrieb Peter Maydell: > These patches fix various compiler and runtime warnings QEMU provokes > on MacOSX 10.8. The first two fix a leak and some deprecated functions > which cause warnings in the system log when QEMU runs. The second > two avoid some functions which cause compile time warnings about > use of functions deprecated in 10.6. > > I believe from my reading of the documentation that these changes > should still work OK on 10.3, but I don't have any way of testing that. > > Andreas: I don't write much ObjC so I don't really have a feel for > what is good coding style. I mostly tried to follow the existing > code (eg "//" style comments) and QEMU's C style (eg 80 cols max). > I'm happy to change any of the indentation or whatever to suit your > preferences. > > With this patch set the only remaining compile time warnings from clang > on 10.8 are about deprecated methods in audio/coreaudio.c. And it doesn't introduce any new issues on v10.5. Thanks, applied to cocoa-for-upstream (with shortened prefix): http://repo.or.cz/w/qemu/afaerber.git/shortlog/refs/heads/cocoa-for-upstream Sorry for the delay, pull coming up. Andreas > Changes v2->v3: > * rebased again > Changes v1->v2: > * rebased and fixed up following recent console changes > > Peter Maydell (4): > ui/cocoa.m: Fix leaks of NSScreen and NSConcreteMapTable > ui/cocoa.m: Avoid deprecated CPS* functions > ui/cocoa.m: Avoid deprecated NSOpenPanel filename method > ui/cocoa.m: Avoid deprecated NSOpenPanel beginSheetForDirectory > > ui/cocoa.m | 52 > 1 file changed, 32 insertions(+), 20 deletions(-) >
Re: [Qemu-devel] [PATCH v3 0/4] ui/cocoa.m: Fix compiler and runtime warnings on 10.8
Ping! -- PMM On 22 April 2013 21:29, Peter Maydell wrote: > These patches fix various compiler and runtime warnings QEMU provokes > on MacOSX 10.8. The first two fix a leak and some deprecated functions > which cause warnings in the system log when QEMU runs. The second > two avoid some functions which cause compile time warnings about > use of functions deprecated in 10.6. > > I believe from my reading of the documentation that these changes > should still work OK on 10.3, but I don't have any way of testing that. > > Andreas: I don't write much ObjC so I don't really have a feel for > what is good coding style. I mostly tried to follow the existing > code (eg "//" style comments) and QEMU's C style (eg 80 cols max). > I'm happy to change any of the indentation or whatever to suit your > preferences. > > With this patch set the only remaining compile time warnings from clang > on 10.8 are about deprecated methods in audio/coreaudio.c. > > Changes v2->v3: > * rebased again > Changes v1->v2: > * rebased and fixed up following recent console changes > > Peter Maydell (4): > ui/cocoa.m: Fix leaks of NSScreen and NSConcreteMapTable > ui/cocoa.m: Avoid deprecated CPS* functions > ui/cocoa.m: Avoid deprecated NSOpenPanel filename method > ui/cocoa.m: Avoid deprecated NSOpenPanel beginSheetForDirectory > > ui/cocoa.m | 52 > 1 file changed, 32 insertions(+), 20 deletions(-) > > -- > 1.7.11.4 > >