Hi Yuv,

> Attached is the fixed patch.  If there are no further bug reports I'll commit
> it soon.
>

Please don't at the current status. There are two points, which you
don't considered:
First, it does not work correctly if you undo/redo a command which
changes the number or order of the images (add/remove image, swap
images and so on). In this case you switch the active state of not
affected images.
Second, I don't think it is a good idea to modify the panorama object
directly (even in the CommandHistory::Undo/Redo). All modifications of
the panorama object should happen inside a PanoCommands, which are
push to the command stack. All other can produce side effects which we
don't consider at the first glance.

Thomas

-- 
You received this message because you are subscribed to the Google Groups 
"Hugin and other free panoramic software" group.
A list of frequently asked questions is available at: 
http://wiki.panotools.org/Hugin_FAQ
To post to this group, send email to hugin-ptx@googlegroups.com
To unsubscribe from this group, send email to 
hugin-ptx+unsubscr...@googlegroups.com
For more options, visit this group at http://groups.google.com/group/hugin-ptx

Reply via email to