hugin-bug-hunters team mailing list archive

[tmodes <1256972@xxxxxxxxxxxxxxxxxx>]

Thanks. Fixed in repository.

** Changed in: panotools
       Status: New => Fix Committed

-- 
You received this bug notification because you are a member of Hugin Bug
Hunters, which is subscribed to Panotools.
https://bugs.launchpad.net/bugs/1256972

Title:
  tiff.c:847: possible bad if test ?

Status in Panorama Tools:
  Fix Committed

Bug description:
  I just ran the static analysis tool cppcheck over the source code 
  of libpano13-2.9.18. It said

  [tiff.c:847] -> [tiff.c:847]: (style) Same expression on both sides of
  '||'.

  Source code is

      if (metadata->xPixelsPerResolution == 0 ||
  metadata->xPixelsPerResolution == 0)

  Maybe

      if (metadata->xPixelsPerResolution == 0 ||
  metadata->yPixelsPerResolution == 0)

  might be better code.

To manage notifications about this bug go to:
https://bugs.launchpad.net/panotools/+bug/1256972/+subscriptions