Re: [PATCH] Overriding automatic syntax detection

[Leonard den Ottolander <leonard den ottolander nl>]

Hello Pavel,

On Thu, 2005-07-07 at 17:35, Pavel Tsekov wrote:
> I've decided to give it a try but it doesn't even compile clean. I applied
> against HEAD.

The patch is called MC_4_6_1_PRE.syntax.patch. Now what do you think
that is supposed to mean?

> Here is the first error I got:
> 
> In file included from ../../mc/edit/picksyntax.c:23:
> ../../mc/edit/../src/dialog.h:196: error: syntax error before '*' token
> ../../mc/edit/../src/dialog.h:196: warning: data definition has no type or storage class

Probably caused by the fact that #includes have been moved around a lot
in HEAD. Discussed this with Roland last night which will help me port
this patch to HEAD.

> ../../mc/edit/picksyntax.c:45: warning: initialization makes pointer from integer without a cast
<snip>

All these might have to do with the undefined reference for N_ I guess.

> ../../mc/edit/picksyntax.c:85: warning: assignment makes pointer from integer without a cast
> ../../mc/edit/picksyntax.c:89: warning: assignment makes pointer from integer without a cast

Well old_syntax_type and option_syntax_type are char*'s and I was under
the impression that g_strdup returns a char* as well, so I'm a bit
puzzled by that warning. Which compiler (version) are you using?

> It is not hard to fix these but please do you homework next time.

You are as inspirational as ever. Please give it a try against PRE
first.

Leonard.

-- 
mount -t life -o ro /dev/dna /genetic/research