winapi_check: Clarify a pair of error messages.

Francois Gouget fgouget at free.fr
Tue Feb 20 08:50:49 CST 2007


Don't use a regular expression if not necessary.
---
 tools/winapi/winapi_documentation.pm |    6 +++---
 1 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/tools/winapi/winapi_documentation.pm b/tools/winapi/winapi_documentation.pm
index 0cbdac1..f41cd49 100644
--- a/tools/winapi/winapi_documentation.pm
+++ b/tools/winapi/winapi_documentation.pm
@@ -137,7 +137,7 @@ sub check_documentation($) {
 		} elsif ($ordinal eq "init") {
 		    $ordinal = 0;
 		} else {
-		    $output->write("documentation: $external_name (\U$module\E.$ordinal) wrong1\n");
+		    $output->write("documentation: invalid ordinal for $external_name (\U$module\E.$ordinal)\n");
 		    next;
 		}
 
@@ -164,8 +164,8 @@ sub check_documentation($) {
 
 		}
 
-		if (!$found && $external_name ne "DllMain" && $ordinal !~ /^0$/) {
-		    $output->write("documentation: $external_name (\U$module\E.$ordinal) wrong2\n");
+		if (!$found && $external_name ne "DllMain" && $ordinal ne "0") {
+		    $output->write("documentation: $external_name (\U$module\E.$ordinal) not declared in the spec file\n");
 		}
 	    }
 	}
-- 
1.4.4.4




More information about the wine-patches mailing list