Thierry Merle wrote:
> kevin liu a écrit :
>   
>> Dear Stoth:
>>     I found that in mt2131.c, function mt2131_set_params(),
>>     line 119 has a little bug:
>>     ^^^^^^^^^^^^^^^^^^^^^
>>     priv->frequency =  (f_lo1 - f_lo2 - MT2131_IF2) * 1000,
>>     should be
>>     priv->frequency =  (f_lo1 - f_lo2 - MT2131_IF2) * 1000;
>>     ______________________________________
>>     And i wonder why Gcc can not check out the fault?????
>>
>> --
>> video4linux-list mailing list
>> Unsubscribe mailto:[EMAIL PROTECTED]
>> https://www.redhat.com/mailman/listinfo/video4linux-list
>>
>>   
>>     
> In ANSI C, this is correct although rarely used like that.
> I suppose it is followed by a line that ends with a semicolon.
> Look at the K&R syntax in BNF here:
> http://www.cs.man.ac.uk/~pjj/bnf/c_syntax.k_r
>
> stat                  : id ':' stat
>                       | 'case' const_exp ':' stat
>                       | 'default' ':' stat
>                       | exp? ';'
>                       | compound_stat
>                       | 'if' '(' exp ')' stat
>                       | 'if' '(' exp ')' stat 'else' stat
>                       | 'switch' '(' exp ')' stat
>                       | 'while' '(' exp ')' stat
>                       | 'do' stat 'while' '(' exp ')' ';'
>                       | 'for' '(' exp? ';' exp? ';' exp? ')' stat
>                       | 'goto' id ';'
>                       | 'continue' ';'
>                       | 'break' ';'
>                       | 'return' exp? ';'
>
>
> exp                   : assignement_exp ( ',' assignment_exp )*
>
> So a statement (stat) must end with ';', it can be an expression (exp)
> that is a list of assignement_exp separated by ','.
>
> No bug in the generated code but subject to discussions, so wrong :)
>
>   
Weird, yeah, I don't like that. I'll put up a patch soon.

Thanks,

- Steve



_______________________________________________
linux-dvb mailing list
[email protected]
http://www.linuxtv.org/cgi-bin/mailman/listinfo/linux-dvb

Reply via email to