Hi Harry,

    I would prefer that it printed out an appropriate message rather
    than crash.


I can only agree with that. I thought that Thomas made a patch for that but I can't find it. Maybe I'm wrong.

The problem is in src/hugin1/hugin/CPDetectorConfig_default.h. It contains the default cp detectors which are used if no saved one are found (e.g. at the first run)

Code:
/** this array saves all default settings */
const struct cpdetector_default default_cpdetectors[]=
{
/* use following order: {description, program name, arguments, type, custom }
   attention: this array have to contain at least one item */

But when build with MAC_SELF_CONTAINED_BUNDLE this array is empty (release 2010.0), but the code assumes, that is contains at least one entry. To add the check code for this special case would make more work than it brings advantage. (When editing the cp detectors settings in the preferences windows, there are checks to assure that at least one entry exists.) In the current mercurial repository the mac part contains 5 entries, so with this version this error should not occur any more.

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 [email protected]
To unsubscribe from this group, send email to 
[email protected]
For more options, visit this group at http://groups.google.com/group/hugin-ptx

Reply via email to