On 19.11.2013 20:49:37, Samuel Thibault wrote: > This is a false positive: what the analyzer doesn't know is that > pset->thread_count is number of elements in the pset->threads queue. > I.e. actual is only ever set to 0 when the pset->threads queue is > empty, and thus no iteration of the loop is done.
[...] > This kind of fix would almost never be a proper one: as I explained > earlier, you have to look with a more global view: start with the > tracing exposed on the analysis webpage, which shows where things > come from: the actual variable. And that's where we know it can't > happen. OK, thanks. I will look at the code again. More carefully this time.