[pulseaudio-discuss] [PATCH 0/3] Fix few issues detected by static analysis

2015-01-12 Thread Boris Egorov
Hello folks. User maksqwe on Russian website habrahabr checked pulseuadio sources with PVS Studio static analyzer and found a few issues. I additionaly checked it with Cppcheck analyzer. Fixes follows. There are more issues, I try to make fixes for them too shortly. Boris Egorov (3): alsa-util

Re: [pulseaudio-discuss] [PATCH 0/3] Fix few issues detected by static analysis

2015-01-13 Thread Peter Meerwald
> User maksqwe on Russian website habrahabr checked pulseuadio sources > with PVS Studio static analyzer and found a few issues. I additionaly > checked it with Cppcheck analyzer. Fixes follows. > > There are more issues, I try to make fixes for them too shortly. applied, thanks the comment mes

Re: [pulseaudio-discuss] [PATCH 0/3] Fix few issues detected by static analysis

2015-01-13 Thread Felipe Sateler
On Tue, Jan 13, 2015 at 8:08 AM, Peter Meerwald wrote: > >> User maksqwe on Russian website habrahabr checked pulseuadio sources >> with PVS Studio static analyzer and found a few issues. I additionaly >> checked it with Cppcheck analyzer. Fixes follows. >> >> There are more issues, I try to make

Re: [pulseaudio-discuss] [PATCH 0/3] Fix few issues detected by static analysis

2015-01-13 Thread Peter Meerwald
> >> User maksqwe on Russian website habrahabr checked pulseuadio sources > >> with PVS Studio static analyzer and found a few issues. I additionaly > >> checked it with Cppcheck analyzer. Fixes follows. > >> > >> There are more issues, I try to make fixes for them too shortly. > > > > applied, th

Re: [pulseaudio-discuss] [PATCH 0/3] Fix few issues detected by static analysis

2015-01-13 Thread Arun Raghavan
On 13 January 2015 at 19:10, Felipe Sateler wrote: > On Tue, Jan 13, 2015 at 8:08 AM, Peter Meerwald wrote: >> >>> User maksqwe on Russian website habrahabr checked pulseuadio sources >>> with PVS Studio static analyzer and found a few issues. I additionaly >>> checked it with Cppcheck analyzer.

Re: [pulseaudio-discuss] [PATCH 0/3] Fix few issues detected by static analysis

2015-01-13 Thread Felipe Sateler
On Tue, Jan 13, 2015 at 10:56 AM, Peter Meerwald wrote: > >> >> User maksqwe on Russian website habrahabr checked pulseuadio sources >> >> with PVS Studio static analyzer and found a few issues. I additionaly >> >> checked it with Cppcheck analyzer. Fixes follows. >> >> >> >> There are more issues

Re: [pulseaudio-discuss] [PATCH 0/3] Fix few issues detected by static analysis

2015-01-13 Thread Felipe Sateler
On Tue, Jan 13, 2015 at 11:09 AM, Arun Raghavan wrote: > On 13 January 2015 at 19:10, Felipe Sateler wrote: >> On Tue, Jan 13, 2015 at 8:08 AM, Peter Meerwald wrote: >>> User maksqwe on Russian website habrahabr checked pulseuadio sources with PVS Studio static analyzer and found a few

Re: [pulseaudio-discuss] [PATCH 0/3] Fix few issues detected by static analysis

2015-01-14 Thread Boris Egorov
On 01/13/2015 05:08 PM, Peter Meerwald wrote: > > the comment messages should have start with an uppercase letter (i.e. > "pacmd: Add missing...") and I should have fixed that; well next time > Oh, I didn't know that. I just looked at a few commit messages before and found that some of them star