On 30 Mai, 17:45, "T. Modes" <thomas.mo...@gmx.de> wrote:
> > 2) API/HEADERS
>
> > Release branches are header-frozen to prevent a change in signature and
> > potential breakage of the scripting wrapper API. An exception may be
> > requested if the underlying motive is important enough. The request must
> > receive the support of a significant majority of developers (coders and
> > scripters) to be granted. Silence is interpreted as supportive of the
> > request.
>
> Here I'm not fully agree. To fix some errors/bugs it may be necessary
> to patch also header files. There are some changes to header files
> possible which does not break the scripting, e.g. add new variables/
> functions. Also it is possible the extend the functions variable list
> (e.g. by using default values for the new values) without breaking the
> scripting.
> So for me this formulation is too strict.
> So I would propose a weaker formulation: Release branches are frozen
> regarding classes/function/namespace names and functions parameters.
I also feel a complete header-freeze would be too drastic. Thomas
points out situations where changes to the header would not affect
hsi. I'd like to add that hsi only wraps headers from panodata,
panotools and algorithms, plus one header in appbase, see below. Other
headers can be modified without any danger to hsi, even if they are
#included by one of the wrapped headers (swig in the mode we're using
does not %include #included headers). While an API specification and
it's reflection in the interface definition is outstanding, it might
be a good idea to put a notice into the headers which are wrapped by
swig. They are listed in the CMake variable hsi_wrappees:
SET(hsi_wrappees
"${HUGIN_BASE_DIR}/appbase/DocumentData.h"
"${HUGIN_BASE_DIR}/panodata/PanoramaData.h"
"${HUGIN_BASE_DIR}/panodata/ImageVariable.h"
"${HUGIN_BASE_DIR}/panodata/PanoImage.h"
"${HUGIN_BASE_DIR}/panodata/ImageVariableGroup.h"
"${HUGIN_BASE_DIR}/panodata/StandardImageVariableGroups.h"
"${HUGIN_BASE_DIR}/panodata/ImageVariableTranslate.h"
"${HUGIN_BASE_DIR}/panodata/Lens.h"
"${HUGIN_BASE_DIR}/panodata/SrcPanoImage.h"
"${HUGIN_BASE_DIR}/panodata/PanoramaVariable.h"
"${HUGIN_BASE_DIR}/panodata/ControlPoint.h"
"${HUGIN_BASE_DIR}/panodata/PanoramaOptions.h"
"${HUGIN_BASE_DIR}/panodata/PanoramaData.h"
"${HUGIN_BASE_DIR}/panodata/Panorama.h"
"${HUGIN_BASE_DIR}/panotools/PanoToolsUtils.h"
"${HUGIN_BASE_DIR}/panotools/PanoToolsInterface.h"
"${HUGIN_BASE_DIR}/panotools/PanoToolsOptimizerWrapper.h"
"${HUGIN_BASE_DIR}/algorithms/PanoramaAlgorithm.h"
"${HUGIN_BASE_DIR}/algorithms/StitcherAlgorithm.h"
"${HUGIN_BASE_DIR}/algorithms/assistant_makefile/
AssistantMakefilelibExport.h"
"${HUGIN_BASE_DIR}/algorithms/basic/CalculateCPStatistics.h"
"${HUGIN_BASE_DIR}/algorithms/basic/CalculateMeanExposure.h"
"${HUGIN_BASE_DIR}/algorithms/basic/CalculateOptimalROI.h"
"${HUGIN_BASE_DIR}/algorithms/basic/CalculateOptimalScale.h"
"${HUGIN_BASE_DIR}/algorithms/basic/CalculateOverlap.h"
"${HUGIN_BASE_DIR}/algorithms/basic/RotatePanorama.h"
"${HUGIN_BASE_DIR}/algorithms/basic/StraightenPanorama.h"
"${HUGIN_BASE_DIR}/algorithms/basic/TranslatePanorama.h"
"${HUGIN_BASE_DIR}/algorithms/control_points/CleanCP.h"
"${HUGIN_BASE_DIR}/algorithms/nona/CalculateFOV.h"
"${HUGIN_BASE_DIR}/algorithms/nona/CenterHorizontally.h"
"${HUGIN_BASE_DIR}/algorithms/nona/ComputeImageROI.h"
"${HUGIN_BASE_DIR}/algorithms/nona/FitPanorama.h"
"${HUGIN_BASE_DIR}/algorithms/nona/NonaFileStitcher.h"
"${HUGIN_BASE_DIR}/algorithms/optimizer/PhotometricOptimizer.h"
"${HUGIN_BASE_DIR}/algorithms/optimizer/PTOptimizer.h"
"${HUGIN_BASE_DIR}/algorithms/panorama_makefile/
PanoramaMakefilelibExport.h"
"${HUGIN_BASE_DIR}/algorithms/point_sampler/PointSampler.h"
)
Kay
--
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
