cvOpenFileStorage: reduce the scope of xml_buf and make sure it's freed...

... before any exceptions occur.

Fix memory leaks appearing when cvOpenFileStorage throws

Merge pull request #3522 from ilya-lavrenov:update_android_toolchain_cmake

update android.toolchain.cmake

Merge pull request #3498 from ilya-lavrenov:omp

Merge pull request #3449 from Ashod:2.4_highgui_trackbar_improvements

ability to enable OpenMP simultaneously with other parallel framework

highgui: Support to change trackbar count in setTrackbarPos and replaced deprecated CreateToolbarEx in Windows.

Merge pull request #3480 from mshabunin:python-test-24

Merge pull request #3488 from jeremyhu:PR-4048

Fix tautologies in calibfilter.cpp which cause a build failure when using -Werror=address with clang-3.5

http://code.opencv.org/issues/4048

Signed-off-by: Jeremy Huddleston Sequoia <jeremyhu@apple.com>

Python test: new options provide local test data paths

Merge pull request #3445 from jet47:get-rid-of-cuda-h

Merge pull request #3447 from a-wi:Fill_ellipse_arc

Fix determining bounding box of the edge collection in FillEdgeCollection().

There is necessary to account also x-ccordinate of the end of the edge when determining bounding box of the edge collection.

get rid of cuda.h usage

Merge pull request #3433 from StevenPuttemans:fix_whitebalance_naming

fix wrong property name and corresponding documentation

Merge pull request #3419 from samyak-268:2.4

Merge pull request #3425 from StevenPuttemans:fix_firewire_buffersize

Merge pull request #3427 from tstellarAMD:2.4-farneback-crash-fix

added extra BUFFERSIZE doc and getter

ocl: Move static oclMat variables into FarnebackOpticalFlow class

Move some static functions into the FarnebackOpticalFlow class as well,
so they can access these new class variables.

oclMat objects cannot be declared statically, because their destructor
depends on the statically defined __module variable from cl_context.cpp.
Since statically defined variables in separate compilation units have
an undefined destruction order there is always the possibility the
__module will be destructed before an oclMat object, which results
in a segfault.

Merge pull request #3421 from SpecLad:ncv-algorithm

NCV.cu: include <algorithm>

It's needed for std::max.

Merge pull request #3412 from StevenPuttemans:ieee1934_camera_set_params

Merge pull request #3414 from SpecLad:vg-stereogc

Merge pull request #3418 from tstellarAMD:2.4-hog-crash-fix

C++ sample code to detect facial features

Merge pull request #3417 from ilya-lavrenov:dox_warn

attemp to fix dox warning

ocl: Fix crash during destruction of gauss_w_lut object in hog.cpp

gauss_w_lut is a statically defined variable of type oclMat.  The oclMat
destructor calls openCLFree() which via getInitializationMutex() accesses
the __module variable which has been statically defined in cl_context.cpp

Since the destruction order of statically defined variables in different
compilation units is undefined, it is possible that __module will
be destructed before gauss_w_lut, which would result in a segfault when
getInitializationMutex() is called while destructing gauss_w_lut.

In order to avoid this issue, we need to make gauss_w_lut a private
member of the HOGDescriptors class so we know it will be destroyed
before __module.

Fix uninitialized memory reads and memory leaks in StereoGC

adding extra buffer parameter

Merge pull request #3401 from SpecLad:optflow-bogus-Warray-bounds

Suppress a bogus -Warray-bounds in modules/superres/src/optical_flow.cpp

This is actually the same warning that the suppression at the bottom
suppresses, but that one doesn't suppress it when using a native compiler
(go figure).

I remove the forced inclusion of precompiled headers into all sources,
since otherwise this workaround doesn't work - the header with the warning
is included before I can use a pragma.

Merge pull request #3322 from StevenPuttemans:fix_pvpapi_interface

Merge pull request #3305 from tSed:opencv-2.4.x/opencv.pc-generation-refactoring

Merge pull request #3294 from mshabunin:fix-ios-warnings-24

Merge pull request #3376 from vriera:2.4

cmake/OpenCVGenPkgconfig.cmake: rework opencv.pc generation

Using absolute path to locate the components in the "Libs:" field of the
*.pc can badly break cross-compilation, especially when building
statically linked objects.

Indeed, pkg-config automatically replaces the '-I...' and '-L...' paths
when the PKG_CONFIG_SYSROOT_DIR and PKG_CONFIG_LIBDIR environment
variables are set [1]. This feature is very helpful and common in
cross-compilation framework like Buildroot [2,3].

When there are absolute paths in the *.pc files, pkg-config won't be
able to do the path substitions for these paths when the afromentioned
environment variables are set.
In such case, since the prefix is the target one, not the sysroot one,
these libraries' abolute paths will point to:
- in the best case: a non-existing file (i.e. these files do not exists
  on the host system;
- at worst: the host system's libraries. This will make the linking
  failed because these host system's libraries will most likely not be
  build for the target architecture [4].

So, this patch replace the components' absolute paths by the form:
  -L<libdir> -l<libname>

This way, the linker will be able to resolve each dependency path,
whatever the kind of objects/build (shared object or static build) it
is dealing with.

Note that for static link, the library order does matter [5]. The order
of the opencv components has been carefully chosen to comply with this
requirement.

Fixes #3931

[1] http://linux.die.net/man/1/pkg-config
[2] http://buildroot.org/
[3] http://git.buildroot.net/buildroot/tree/package/pkgconf/pkg-config.in
[4] http://autobuild.buildroot.net/results/e8a/e8a859276db34aff87ef181b0cce98916b0afc90/build-end.log
[5] http://stackoverflow.com/questions/45135/linker-order-gcc

Signed-off-by: Samuel Martin <s.martin49@gmail.com>
---
Note: this patch properly applies on top of the master branch, though it
      has been written on top of the 2.4 branch.

Merge pull request #3382 from StevenPuttemans:fix_3982

Merge pull request #3364 from mshabunin:doxygen-docs-24

fixing SVM negative value rounding

fixing cap_pvpapi interface and making sure both AVT Manta and Proscilla
series are supported. Testing this with both cams for Windows and Linux
exhaustively.

Optimizing memory footprint by removing unused calls.

Adapted with the input of Shai

Added small example that illustrates how it should work.

superres: Fix \ 1\ 1return value VideoFrameSource_GPU

superres module fails to compile with the following error messages:

[100%] Building CXX object modules/superres/CMakeFiles/opencv_superres.dir/src/super_resolution.cpp.o
/opencv-2.4.10/modules/superres/src/frame_source.cpp: In function 'cv::Ptr<cv::superres::FrameSource> cv::superres::createFrameSource_Video_GPU(const string&)':
/opencv-2.4.10/modules/superres/src/frame_source.cpp:263:16: error: expected type-specifier before 'VideoFrameSource'
/opencv-2.4.10/modules/superres/src/frame_source.cpp:263:16: error: could not convert '(int*)operator new(4ul)' from 'int*' to 'cv::Ptr<cv::superres::FrameSource>'
/opencv-2.4.10/modules/superres/src/frame_source.cpp:263:16: error: expected ';' before 'VideoFrameSource'
/opencv-2.4.10/modules/superres/src/frame_source.cpp:263:41: error: 'VideoFrameSource' was not declared in this scope
/opencv-2.4.10/modules/superres/src/frame_source.cpp:264:1: error: control reaches end of non-void function [-Werror=return-type]
cc1plus: some warnings being treated as errors
make[3]: *** [modules/superres/CMakeFiles/opencv_superres.dir/src/frame_source.cpp.o] Error 1
make[3]: *** Waiting for unfinished jobs....

This is caused because the return value of the \ 1createFrameSource_Video_GPU function should be a VideoFrameSource_GPU object.

iOS framework build script changes from master branch

Fixed iOS framework compilation warnings

iOS framework build script changes

- buildbot process support (break build in case of errors)
- disabled some warnings in cmake script

Merge pull request #3336 from BKNio:putText_develop2

fixed issue with malformed UTF-8 string;

ocl: Change static variable order in cl_context.cpp to avoid crashes during destruction

ContextImpl::currentContext contains a reference to one of the
DeviceInfoImpl objects from:

static std::vector<DeviceInfoImpl> global_devices;

ContextImpl::currentContext is destroyed in the destructor
for the statically defined object __module, and relies on its
DeviceInfoImpl reference to query some hardware features while
being destroyed.

This means that we need to ensure that the global_devices vector is
destroyed affter __module, otherwise ContextImpl::currentContext's
DeviceInfoImpl reference will no longer be valid when __module is
destroyed.

Since these variables are all confined to a single compilation unit,
they will be destruct from bottom to top, so we need to make sure
that __module is the bottom definition so it can be destroyed first.

iOS: fix crash from overrelease in UIImageToMat

viz: fixed memory leak, issue 3961

fix installation layout for debian packages:

Install symlinks to shared libraries as a part of development package,
not runtime package.

It is default behavior for debian packages.

Fix test name.

TIFF loader: Allocate large enough buffer when (bpp * ncn) > 8.

TIFF loader: Pass buffer size to read functions.

replace not ascii and not cyrillic symbols with '?';

add test for putText;

fix warning;

minor fixes;

Merge pull request #3362 from mshabunin:fix-drawing-24

Merge pull request #3363 from asmorkalov:qnx_support

Merge pull request #3366 from jet47:fix-gpu-cuda-7.0

Merge pull request #3356 from jet47:ctest-support

minor fix for StereoCSBP data cost compute kernel and test

increase epsilons for some tests, which functions use floating point arithm

avoid pointer arithmetic on register memory in color conversion

Basic doxygen documentation support

- updated existing Doxyfile.in
- added corresponding cmake instructions
- added some specific files (layout, icon)
- clean existing doxygen warnings

Conflicts:
CMakeLists.txt
doc/CMakeLists.txt
modules/core/include/opencv2/core.hpp
modules/core/include/opencv2/core/base.hpp
modules/core/include/opencv2/core/cuda.inl.hpp
modules/core/include/opencv2/core/mat.hpp
modules/core/include/opencv2/core/matx.hpp
modules/core/include/opencv2/core/types.hpp
modules/flann/include/opencv2/flann/lsh_table.h
modules/imgproc/include/opencv2/imgproc.hpp

Fixes for QNX 6.6 Neitrino support.

Fixed highgui drawing test

- changed image comparison technique
- removed old assert statements

determine test category (Public vs Extra) based on module location

add CTest targets for sanity tests

disable CTest target for opencv_test_viz:

opencv_test_viz creates windows and waits until user close them,
it is not appropriate for automatic testing

rewrite add_test command for accuracy tests:

* use new syntax introduced in CMake 2.8.0
* set OPENCV_TEST_DATA_PATH enviroment property

turn on testing support in root binary directory

Merge pull request #3350 from Trundle:fix_some_tiff_crashes

TIFF loader: Pass buffer size to read functions.

TIFF loader: Allocate large enough buffer when (bpp * ncn) > 8.

Fix test name.

Merge pull request #3341 from jet47:deb-package-lib-symlinks

Merge pull request #3346 from Nerei:fixed_memory_leak_in_viz

viz: fixed memory leak, issue 3961

Merge pull request #3340 from AdLantis:2.4

fix installation layout for debian packages:

Install symlinks to shared libraries as a part of development package,
not runtime package.

It is default behavior for debian packages.

iOS: fix crash from overrelease in UIImageToMat

Merge pull request #3337 from tstellarAMD:2.4-ocl-context-cleanup-fix

ocl: Change static variable order in cl_context.cpp to avoid crashes during destruction

ContextImpl::currentContext contains a reference to one of the
DeviceInfoImpl objects from:

static std::vector<DeviceInfoImpl> global_devices;

ContextImpl::currentContext is destroyed in the destructor
for the statically defined object __module, and relies on its
DeviceInfoImpl reference to query some hardware features while
being destroyed.

This means that we need to ensure that the global_devices vector is
destroyed affter __module, otherwise ContextImpl::currentContext's
DeviceInfoImpl reference will no longer be valid when __module is
destroyed.

Since these variables are all confined to a single compilation unit,
they will be destruct from bottom to top, so we need to make sure
that __module is the bottom definition so it can be destroyed first.

Merge pull request #3331 from tSed:opencv-2.4.x/fix-x86-PIC-compilation

core: fix x86 PIC code compilation

This bug was triggered by Buildroot autobuilders [1,2], causing this
kind of failures [3,4]:

  [ 14%] Building CXX object modules/core/CMakeFiles/opencv_core.dir/src/system.cpp.o
  /home/test/autobuild/instance-0/output/build/opencv-2.4.10/modules/core/src/system.cpp: In function '(static initializers for /home/test/autobuild/instance-0/output/build/opencv-2.4.10/modules/core/src/system.cpp)':
  /home/test/autobuild/instance-0/output/build/opencv-2.4.10/modules/core/src/system.cpp:280:10: error: inconsistent operand constraints in an 'asm'
  make[3]: *** [modules/core/CMakeFiles/opencv_core.dir/src/system.cpp.o] Error 1

[1] http://buildroot.org/
[2] http://autobuild.buildroot.org/
[3] http://autobuild.buildroot.org/?reason=opencv-2.4.10
[4] http://autobuild.buildroot.org/results/483/4838285b25d6293a5cf0bb9eadd5040a7c75d766/build-end.log

Signed-off-by: Samuel Martin <s.martin49@gmail.com>

Merge pull request #3265 from BKNio:putText_develop

Merge pull request #3329 from taka-no-me:patch-1

Merge pull request #3300 from rohitrawat:bug_3787_docs_NAryMatIterator

Merge pull request #3327 from JonasVautherin:patch_cmake_android_r10

silence cmake 3.0+ warnings on LOCATION attribute

Patch for NDK r10

fix issue with font;

Merge pull request #3307 from ilya-lavrenov:ffmpeg_rtsp

TCP instead of UDP

Merge pull request #3298 from a-wi:DShow_capture

Merge pull request #3309 from nisargthakkar:Thickness1NoLineCV_AA

Merge pull request #3311 from BarneyBear:2.4-drawMatches-flag-fix

Merge pull request #3316 from nisargthakkar:Bug3887

Merge pull request #3319 from Trundle:tiff_infinite_rowsperstrip

Don't allocate lots of memory when reading TIFFs with infinite rows per strip.

Some TIFF images consist of only one strip. The magic value 2**32-1 for the
"rows per strip" tag reflects that fact, effectively meaning "infinite".

Made some changes specefict to 4 channel images. Fixed step value for tptr

Fixed doc for Bug #3887

Fixed doc for Bug #3887

Fixes  #3910, flags in _prepareImgAndDrawKeypoints are now binary OR'ed instead of added.

Fixing bug in NAryMatIterator documentation(Bug #3787).

Fixed the non-working code example for NAryMatIterator.
The documentation of NAryMatIterator in file:
modules/core/include/opencv2/core/core.hpp has a working example which
was used for these edits.

Fixed getting CV_CAP_PROP_FOURCC property value

Save selected FCC in order to return proper CV_CAP_PROP_FOURCC property value (currently FCC for RGB24 is always returned).