Archived issue #0023488
Make video capture functionality in TKOpenGl optional
Description
#0020035 introduce video capture capabilities to TKOpenGl.
However this functionality is limited to Windows platform (used Video For Windows) and rarely used.
Some remarks from R.Lygin ~13699:
>Just came up to write you about this issue detected once 6.3.0 has been out.
>It seems that with introduction of avi export
>(and/or maybe other new stuff),
>TKV3d or TKOpenGl now depend on greater amount of system libraries.
>You can see this when running an app under debugger in MSVS.
>
>Obviously this increases load time and memory footprint.
>As this functionality will really rarely be used,
>the benefit does not justify the cost.
>Perhaps deferred loading would be a better option.
>Have you considered this?
This is suggested to make this functionality optional as it done for other cases (FreeImagePlus, gl2ps).
However this functionality is limited to Windows platform (used Video For Windows) and rarely used.
Some remarks from R.Lygin ~13699:
>Just came up to write you about this issue detected once 6.3.0 has been out.
>It seems that with introduction of avi export
>(and/or maybe other new stuff),
>TKV3d or TKOpenGl now depend on greater amount of system libraries.
>You can see this when running an app under debugger in MSVS.
>
>Obviously this increases load time and memory footprint.
>As this functionality will really rarely be used,
>the benefit does not justify the cost.
>Perhaps deferred loading would be a better option.
>Have you considered this?
This is suggested to make this functionality optional as it done for other cases (FreeImagePlus, gl2ps).
Additional information
Video capture capabilities (implemented only for Windows platform) is not build by default. To build OCCT with this functionality HAVE_VIDEOCAPTURE should be defined in compiler options (directly in project settings or in CSF_DEFINES environment variable which defined in "env.bat").
Public activity
3 archived notes
Participants are labeled by their role within this record.
Patch is ready for review in CR23488 branch
Reviewed, please test. Kirill, please provide documentation for release notes
Dear Commenter 1,
Branch CR23488 (and products from GIT master) was compiled on Linux and Windows platforms and tested.
Regressions:
Not detected
Improvements:
Not detected
Testing cases:
Not needed
Branch CR23488 (and products from GIT master) was compiled on Linux and Windows platforms and tested.
Regressions:
Not detected
Improvements:
Not detected
Testing cases:
Not needed