On Thu, Dec 09, 2010 at 10:01:05AM +0100, Petter Reinholdtsen wrote:

> -            for (g=1; g < (vidVect.size()-1); g++) {
> +            for (size_t g=1; g < (vidVect.size()-1); g++) {

Brainflippin piece of code isn't it ?
It skips first and last element from the scan. Why ?

It _seems_ the loop is there to find duplicates of the just-added
device, which would explain why skippin the last (the last is 
what it's comparin to) but doesn't explain skippin the first.

Just trying to give hints to whoever is willing to clean that code
up. The warnings removal patch is correctly not changing semantic.

--strk; 

  ()   Free GIS & Flash consultant/developer
  /\   http://strk.keybit.net/services.html

_______________________________________________
Gnash-dev mailing list
Gnash-dev@gnu.org
http://lists.gnu.org/mailman/listinfo/gnash-dev

Reply via email to