Jump to content

Recommended Posts

Posted

Boas, é a minha de de pedir ajuda aqui ao pessoal.


uint16_t calcCRC(uint8_t *frame,int size)
{
uint16_t CRCVal=0;
int i;
for( i = 0 ; i < size ; i++ )
{
	CRCVal = CRCTABLE[( CRCVal ^= frame[ i ] ) & 0xFF ] ^ ( CRCVal >> 8 );
}
return CRCVal;
}

Esta função calcula o CRC de 16 bits segundo as normas ISO 3309, CCITT V42 e CCITT X25. Para tal usa uma tabela predefinida:

const uint16_t CRCTABLE[256]=
{
   0xF078,0xE1F1,0xD36A,0xC2E3,0xB65C,0xA7D5,0x954E,0x84C7, ... 
};

O compilador dá-me um warning na linha:

CRCVal = CRCTABLE[( CRCVal ^= frame[ i ] ) & 0xFF ] ^ ( CRCVal >> 8 );

 

Com a descrição: "operation on 'CRCVal' may be undefined".

Alguém vislumbra o porquê deste warning, e como o limpar?

Posted

CRCVal = CRCTABLE[( CRCVal ^= frame[ i ] ) & 0xFF ] ^ ( CRCVal >> 8 );

Estás a tentar mudar o valor de CRCVal duas vezes na mesma instrução (mais propriamente: sem um "sequence point" de intervalo).

Muda a segunda atribuição para uma instrução isolada

CRCVal ^= frame[ i ];
CRCVal = CRCTABLE[CRCVal & 0xFF ] ^ ( CRCVal >> 8 );

What have you tried?

Não respondo a dúvidas por PM

A minha bola de cristal está para compor; deve ficar pronta para a semana.

Torna os teus tópicos mais atractivos e legíveis usando a tag CODE para colorir o código!

Posted

... o código foi retirado directamente do manual do fornecedor ...

O fornecedor podia (devia!) aprender contigo a ligar os warnings do compilador 👍

What have you tried?

Não respondo a dúvidas por PM

A minha bola de cristal está para compor; deve ficar pronta para a semana.

Torna os teus tópicos mais atractivos e legíveis usando a tag CODE para colorir o código!

Create an account or sign in to comment

You need to be a member in order to leave a comment

Create an account

Sign up for a new account in our community. It's easy!

Register a new account

Sign in

Already have an account? Sign in here.

Sign In Now
×
×
  • Create New...

Important Information

By using this site you accept our Terms of Use and Privacy Policy. We have placed cookies on your device to help make this website better. You can adjust your cookie settings, otherwise we'll assume you're okay to continue.