Results 538 comments of Amit Dovev

CMake and Autotools should behave similarly, otherwise you make supporting the software more difficult.

Due to the differences between the builds, for Tesseract at least I would prefer that we support just one build system for Linux, macOS,, and the BSDs.

#4026 is one more example for a different behavior of the Autotools build vs. the CMake build.

Which data files did you use? There are three repos of data files: `tessdata`, `tessdata`_fast, `tessdata_best`. Also, there are `eng` and `Latin` data files. You can try and see if...

https://en.cppreference.com/w/cpp/language/attributes/maybe_unused

https://groups.google.com/g/tesseract-ocr/c/PMHq6YSpRRE @zdenop, In the linked thread you confirmed that `SetRectangle()` is not working as expected.

>I also made `FindLines` public so it can be run directly without running `Recognize` or `AnalyseLayout` first (running `AnalyseLayout` would result in paragraph recognition being run twice). So paragraph detection...

>Why not use leptonica solution? Because it is not necessary. Tesseract does it anyway.

You are right that my suggestion changes the current behavior, so it's not a good idea. I still think we should not expose `FindLines` just to make this use case...

@zdenop, After reading @Balearica's answer to your question, do you object to merging this PR? @stweil, can we merge it?