From: <ad...@us...> - 2010-11-06 13:54:37
|
Revision: 1171 http://jtidy.svn.sourceforge.net/jtidy/?rev=1171&view=rev Author: aditsu Date: 2010-11-06 13:54:31 +0000 (Sat, 06 Nov 2010) Log Message: ----------- added extra condition (from the C code) for calling errorSummary in Tidy compatibility mode - fixes 3 test cases Modified Paths: -------------- branches/CodeUpdateAndJava5/src/main/java/org/w3c/tidy/Tidy.java Modified: branches/CodeUpdateAndJava5/src/main/java/org/w3c/tidy/Tidy.java =================================================================== --- branches/CodeUpdateAndJava5/src/main/java/org/w3c/tidy/Tidy.java 2010-11-06 13:04:54 UTC (rev 1170) +++ branches/CodeUpdateAndJava5/src/main/java/org/w3c/tidy/Tidy.java 2010-11-06 13:54:31 UTC (rev 1171) @@ -590,8 +590,9 @@ o.flush(); } - if (!configuration.isQuiet()) - { + // Tidy only shows the error summary when there are errors and/or warnings, + // but that can miss e.g. badlayout messages + if ((!configuration.isTidyCompat() || parseErrors + parseWarnings > 0) && !configuration.isQuiet()) { this.report.errorSummary(lexer); } This was sent by the SourceForge.net collaborative development platform, the world's largest Open Source development site. |