hugin-devs team mailing list archive

[tmodes <1575120@xxxxxxxxxxxxxxxxxx>]

No, in C++ there are also double variants of abs.
But it is differently used by different compilers.
Fixed in repository.

Nevertheless, who needs such small angles for rotating without a GUI?

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

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

Title:
  pano_modify --rotate does not detect values < abs(1) on hugin 2016.0

Status in Hugin:
  Fix Committed

Bug description:
  If on command line a pano_modify --rotate parameter is given,
  pano_modify checks for valid values with (pano_modify.cpp, line 745):

  if (abs(yaw) + abs(pitch) + abs(roll) > 0.0)

  If I'm not mistaken, abs() expects ints and the if-case will therefore
  return false, if no parameter is > abs(1)


  System Information:
  Betriebssystem: Linux 4.2.0-23-generic x86_64
  Architektur: 64 bit
  Freier Speicher: 140012862658896 kiB

  Hugin
  Version: 2016.0.0.3b4e2790cb90
  Ressourcen-Pfad: /usr/share/hugin/xrc/
  Datenpfad: /usr/share/hugin/data/
  Hugins Kamera- und Objektivdatenbank: /home/xxx/.hugindata/camlens.db
  Multi-Threading mittels C++11 std::thread und OpenMP

  Bibliotheken
  wxWidgets: wxWidgets 3.0
  wxWidgets Library (wxGTK port)
  Version 3.0.0 (Unicode: wchar_t, debug level: 1),
  compiled at Dec  2 2013 15:56:33

  Runtime version of toolkit used is 2.24.
  Compile-time GTK+ version is 2.24.22.

  libpano13: 2.9.19 
  Boost: 1.54.0
  Exiv2: 0.23
  SQLite3: 3.8.2
  Vigra: 1.10.0
  LittleCMS2: 2.5

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