Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Move AVX / SSE messages to function PrintVersionInfo (crash fix) #625

Merged
merged 1 commit into from
Dec 28, 2016

Conversation

stweil
Copy link
Member

@stweil stweil commented Dec 28, 2016

This information is not needed for normal runs, so it is sufficient
to show it on request (like versions and OpenCL information).

This also fixes a crash caused by undefined order of global constructors:

When the global variable SIMDDetect::detector is initialized before the
global variable debug_file, the first tprintf call in simddetect.cpp
crashes because of a NULL pointer in debug_file. This was only seen when
running with a shared library (libtesseract.so).

Signed-off-by: Stefan Weil sw@weilnetz.de

This information is not needed for normal runs, so it is sufficient
to show it on request (like versions and OpenCL information).

This also fixes a crash caused by undefined order of global constructors:

When the global variable SIMDDetect::detector is initialized before the
global variable debug_file, the first tprintf call in simddetect.cpp
crashes because of a NULL pointer in debug_file. This was only seen when
running with a shared library (libtesseract.so).

Signed-off-by: Stefan Weil <sw@weilnetz.de>
@zdenop zdenop merged commit e6ccfb2 into tesseract-ocr:master Dec 28, 2016
@egorpugin
Copy link
Contributor

This broke windows build.

@egorpugin
Copy link
Contributor

More precisely - dll build.
I've fixed it. Please, be careful with static variables.

@stweil stweil deleted the lstm branch December 28, 2016 21:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants