hugin-devs team mailing list archive

[tmodes <679917@xxxxxxxxxxxxxxxxxx>]

The patch needs some more polishing (I had no time to compile, but some comments from have a look on it):
* The timer is not correctly freed at the end.
* Don't use local variables beginning with m_ (these should only used for member variables to get a better overview)
* Status reports should go to stdout and not to stderr (and no blabla, what should this?? bad style). Also fprintf can make problems with Windows. There are better alternatives: e.g. using cout, DEBUG_TRACE,
* The 2 conversion of the filename should not be necessary. 
* It needs to be checked if the path in the project file is correct (the cwd can be an other directory than the dir with the image files, e.g. user navigated to lens or mask files.)
* Don't autosave if the project does not contain any image.
* Maybe the EVT_TIMER macro would be better suit to the existing code than using the eventhandler.

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

Status in Hugin - Panorama Tools GUI: Confirmed

Bug description:
Hugin just crashed on me. Pre 0.7.0 this happened so often that I knew I had to save often, or risk losing my work. 

So far this is the first since I went to 0.7.0. Now 0.8.0 crashed on me.... However I'm most pissed that a pto file got saved to run "nona", but this gets (I think) cleaned up after running nona. So now I'm left with a project file with hours of work missing. 

PLEASE write a backup pto file where I can find it, or mention "this project had unsaved changes when hugin crashed, load the backup file?" when I start it. (then I don't need to know where the backup is stored...).