Comparing String with enum in CMScorer.java

[brodecva]

Hi, @tomas-knap , in the statement
// special context blocks does not count towards the maximum number of context blocks to be // considered if (ctx.getType().equals(TContext.TableContextType.CAPTION) || ctx.getText().equals(TContext.TableContextType.PAGETITLE)) { // FIXME: Comparing String to Enum (maybe PAGETITLE.getText() was meant?) } else { countContextBlocks++; }

in the CMScorer.java the comparison between ctx.getText() and TContext.TableContextType.PAGETITLE is evidently wrong, as you compare string with an enum. This is bound to always return false. I assume that you actually want to compare the ctx.getText() with TContext.TableContextType.PAGETITLE.getText(), but I was not sure enough to fix it myself. Please look into it. Thx!

[tomas-knap]

Thanks, will look into that!