[GRASS-dev] Re: i.modis.qc

[Yann Chemin]

Hello again Glynn,

maybe it is wiser indeed to change to ">>=" for platform/compiler
stability as it forces the variable to be updated.
Somehow it is not necessary on my system (Debian/Sid).

Thanks,
Yann

2008/10/24 Yann Chemin <yann.chemin at gmail.com>:
> Hello Glynn,
>
> It should be standard bit shift ">>", so as to push the binary bits of
> interest at the right side to be used for classification.
> I do not get those warnings, and it processes as expected. Anything
> special about your compilation/compiler maybe?
>
> Yann
>
>
>
> 2008/10/24 Glynn Clements <glynn at gclements.plus.com>:
>>
>> i.modis.qc currently generates a number of warnings:
>>
>> qc250b.c:14: warning: statement with no effect
>> qc250c.c:20: warning: statement with no effect
>> qc250d.c:12: warning: statement with no effect
>> qc250e.c:12: warning: statement with no effect
>> qc250f.c:12: warning: statement with no effect
>> qc500c.c:22: warning: statement with no effect
>> qc500d.c:12: warning: statement with no effect
>> qc500e.c:12: warning: statement with no effect
>>
>> These all relate to code such as:
>>
>>    qctemp >> 2;                /*bits [2-3] become [0-1] */
>>
>> Is this supposed to be:
>>
>>    qctemp >>= 2;
>> or:
>>    pixel >>= 2;
>>
>> instead?
>>
>> --
>> Glynn Clements <glynn at gclements.plus.com>
>>
>
>
>
> --
> Yann Chemin
> International Rice Research Institute
> Office: http://www.irri.org/gis
> Perso: http://www.freewebs.com/ychemin
> YiKingDo: http://yikingdo.unblog.fr/
>



-- 
Yann Chemin
International Rice Research Institute
Office: http://www.irri.org/gis
Perso: http://www.freewebs.com/ychemin
YiKingDo: http://yikingdo.unblog.fr/