Silent ABI change in 3.04.x

[innir]

Hi,

if seems, that moving PageIterator* AnalyseLayout() to api/baseapi.h[1] causes a silent ABI breakage. Could you please move it back to api/baseapi.cpp? Please see the corresponding Debian bugs for reference [2], [3].
I could prepare a PR if you prefer.

Thanks,
Philip

[1] https://github.com/tesseract-ocr/tesseract/blob/master/api/baseapi.h#L500
[2] https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=816857
[3] https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=815056#24

[jbreiden]

See also the tail end of https://groups.google.com/forum/#!topic/tesseract-dev/8e0F2cK2YzU

[zdenop]

Please create PR. Thanks.

[innir]

Here you go: PR #259

[zdenop]

thanks.

[jbreiden]

Please make sure this reaches the 3.04 branch.

[zdenop]

done.

Zdenko

On Tue, Mar 8, 2016 at 5:03 PM, jbreiden [email protected] wrote:

Please make sure this reaches the 3.04 branch.

—
Reply to this email directly or view it on GitHub
#254 (comment)
.