Re: [patch] C99 restrict support

[Roger Leigh <rleigh whinlatter ukfsn org>]

Damon Chaplin <damon karuna uklinux net> writes:

> On Thu, 2005-01-27 at 16:12, Roger Leigh wrote:

>> *** Rebuilding template files ***
>> cd . && gtkdoc-mktmpl --module=uterm
>> WARNING: No declaration for: uterm_code_table_get_start_char
>> WARNING: No declaration for: uterm_code_table_get_num_chars
>> WARNING: No declaration for: uterm_code_table_get_map_start_char
>> WARNING: No declaration for: uterm_code_table_get_map_num_chars
>> WARNING: No declaration for: uterm_code_table_map_get_start_char
>> WARNING: No declaration for: uterm_code_table_map_get_num_chars
>> WARNING: No declaration for: uterm_code_table_map_get_map_start_char
>
>> It's the "WARNING: No declaration for: xxx" messages that worry me
>> most (the others are due to the docs being incomplete).  What is
>> causing these warnings?
>
> I think you need to fix gtkdoc-scan.in as well. That does the initial
> scanning of the header files.

Ah.  It looks like that was a separate problem (not checking for
unsigned).  This patch fixes that:

--- /var/tmp/gtkdoc-scan	2005-01-27 19:48:29.000000000 +0000
+++ gtkdoc-scan	2005-01-27 21:07:43.000000000 +0000
@@ -419,7 +419,7 @@
 		$decl = $';
 		if ($previous_line !~ m/^\s*G_INLINE_FUNC/
 		    && $previous_line !~ m/^\s*static\s+/) {
-		    if ($previous_line =~ m/^\s*(extern)?\s*((const\s*|G_CONST_RETURN\s*)?\w+)(\s*\*+)?\s*$/) {
+		    if ($previous_line =~ m/^\s*(extern)?\s*((const\s*|G_CONST_RETURN\s*|unsigned\s*)?\w+)(\s*\*+)?\s*$/) {
 			$ret_type = $2;
 			if (defined ($4)) { $ret_type .= " $4"; }
 #			print "Function: $symbol, Returns: $ret_type\n";


Should I file this in bugzilla as well, or is here sufficient?


Regards,
Roger

-- 
Roger Leigh
                Printing on GNU/Linux?  http://gimp-print.sourceforge.net/
                Debian GNU/Linux        http://www.debian.org/
                GPG Public Key: 0x25BFB848.  Please sign and encrypt your mail.