akorentlab / leptonica

Automatically exported from code.google.com/p/leptonica
0 stars 0 forks source link

bestfirst.cpp has empty for loop body. #71

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?
1. compile bestfirst.cpp
2.
3.

What is the expected output? What do you see instead?

clean compile.

Instead, warning of for loop w/o body:

      ASSERT_HOST(!current_choices->empty());
      BLOB_CHOICE_IT choice_it(current_choices);
      for (choice_it.mark_cycle_pt(); !choice_it.cycled_list() &&
           merged_choice->rating() > choice_it.data()->rating();
           choice_it.forward());
        choice_it.add_before_stay_put(merged_choice);

What version of the product are you using? On what operating system?
1.69, OS X 10.8

Please provide any additional information below.
I assume the semi-colon is misplaced and will include add_before... in loop.

Original issue reported on code.google.com by stephen....@gmail.com on 25 Dec 2012 at 4:57

GoogleCodeExporter commented 9 years ago
oops - this is actually from tesseract code body.

Original comment by stephen....@gmail.com on 25 Dec 2012 at 4:58

GoogleCodeExporter commented 9 years ago
The indentation is incorrect, not the semicolon.
Replaced with {} in 3.03.

Original comment by r...@google.com on 2 Jan 2013 at 6:53

GoogleCodeExporter commented 9 years ago
This is fixed in the tessseract 3.03 release.

Original comment by dan.bloo...@gmail.com on 11 Jul 2013 at 9:27