[PATCH 0/3] rnetclient 2014, compatibilidade

[Thadeu Lima de Souza Cascardo]

On Thu, Mar 06, 2014 at 10:07:06AM -0300, Alexandre Oliva wrote:
> O primeiro patch corrige um warning do GCC quando tentei compilar o
> rnetclient em x86_64.
> 
> O segundo corrige um bug no cálculo do comprimento de mensagens,
> quando o segundo byte era um caractere com o oitavo bit ativo.
> 
> O terceiro é minha tentativa de atualizar o rnetclient para o formato
> de arquivo do IRPF 2014.  Foi suficiente para que o rnetclient
> chegasse a enviar a declaração pra Receita Federal, mas recebi
> resposta de erro.  Será que mudaram algo no protocolo?
> 

Acho que a única mudança foi mesmo no tamanho do cabeçalho. Como você
reparou depois (e eu não tinha notado também), dados_val seguia usando o
tamanho do cabeçalho de 2013.

Eu apliquei seus patches 1, 2 e 4. Fiquei um pouco indeciso quanto ao
último, mas decidi aplicá-lo mesmo assim. O valor 10 é para antecipar
alguns bytes que poderão ser adicionados logo em seguida, certo?

Quanto ao terceiro e mais importante patch, eu havia pela manhã de
ontem, escrito boa parte dele. Mas alguns outros trechos são
interessantes e valem outros patches. E aproveitei algumas mudanças que
você fez, como o uso de macros pro tamanho do cabeçalho, mas algumas das
suas idéias ainda valem outros patches (como calcular o início e final
do cabeçalho que devemos enviar a partir dos campos utilizados).

Uma diferença importante na minha abordagem é que mantive suporte à
declaração de 2013. Na verdade, sei que o código não funciona hoje para
declarações anteriores, que acho importante suportar. Pode dar algum
trabalho a mais, mas gostaria de fazer um esforço. Vou tentar suportar
os últimos 5 anos a cada ano.

No entanto, caso o esforço seja muito grande e por falta de oportunidade
de testes (não vou retificar cada um dos 5 anos anteriores), mudei a
numeração do release para usar o ano como versão maior. Portanto, já
temos uma versão 2014.0, que sabemos que não funciona, já que o
cabeçalho enviado não tem o tamanho correto.

Vou realizar testes este final de semana e farei um release na
segunda-feira.

Novos testes são bem-vindos.

Abraços e obrigado pelos patches, pelo irpf-livre e pelo incentivo.
Cascardo.

> ---
> 
> Alexandre Oliva (3):
>       Silence compiler warning in 64-bit mode
>       Convert chars to len with unsigned buffer
>       Update header parser for IRPF2014
> 
> 
>  decfile.c    |    9 +++++++--
>  rnetclient.c |   12 ++++++++----
>  2 files changed, 15 insertions(+), 6 deletions(-)
> 
> --
> Alexandre Oliva, freedom fighter    http://FSFLA.org/~lxoliva/
> You must be the change you wish to see in the world. -- Gandhi
> Be Free! -- http://FSFLA.org/   FSF Latin America board member
> Free Software Evangelist     Red Hat Brazil Toolchain Engineer
>