eLobato/ASAFACMS — pull request #1 by alfongj (2013-04-21 09:31:22+00:00)
Title: 2 small fixes

Al mirar en detalle el código, he visto que mandas todos los números como caracteres o como valores numéricos de 1B, y por tanto no hace falta tocar el Endianness como te dije por twitter. Me había confundido suponiéndolo al ver que en el cliente Java especificabas el orden de los ByteBuffers como Little Endian.

De todas formas, eso último no haría falta: el endianness solo afecta al enviar enteros representados de manera numérica y que ocupen varios bytes, es decir, shorts, ints, longs... Y por ello te he quitado las líneas `bb.order(ByteOrder.LITTLE_ENDIAN);` que no afectaban al output.

Si sí enviases enteros de más de 1B, entonces la convención siempre es enviarlos en BigEndian y al recibirlos en el servidor, pasártelos al Endianness de tu PC con ntohs/ntohl (en C), o no hacer nada en caso de Java, ya que siempre es BigEndian.

Ya que estaba, he corregido un par de bugs pequeños que te detallo en el commit

---
Changes:
1. In client.java: replaced several instances of read(byte[]) with
readFully(byte[]) as the first doesn't guarrantee to read the full
array, which might lead to unexpected results.
2. In README: the Java CLI execution line had a typo

Also commented all the ByteBuffer.order(LITTLE_ENDIAN) lines, as they
don't serve any purpose there, and might lead to confusion.