Hi Thomas,
2010/5/27 T. Modes <[email protected]>
> 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 are completely right! Thanks for reminding me what I configured myself.
:-)
Harry
--
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
