vector deserialisation (read)

Anatoscope / sofa

Real-time multi-physics simulation with an emphasis on medical simulation.

https://www.sofa-framework.org

2 stars 1 forks source link

vector deserialisation (read) #21

Open matthieu-nesme opened 7 years ago

matthieu-nesme commented 7 years ago

Pourquoi spécialiser sur unsigned char mais pas sur char ? Je pense qu'on peut avoir une seule implém générique avec un petit template qui va bien.

Veut-on garder la désérialisation de 'A-B'? qui ne fonctionne pour pour int/unsigned int (je crois)?

Suggested labels:

thomas-lemaire commented 7 years ago

La spécialisation "unsigned char" permet de considérer les valeurs comme des entiers (cast vers int) C'est moche, c'est sofa "historique"

j'imagine que supprimer la dé-sérialisation 'A-B' va casser des exemples...

matthieu-nesme commented 7 years ago

Il faut conserver le cast pour les unsigned char, mais le faire aussi pour les char, et mutualiser le code (pas besoin de dupliquer tout le code juste pour caster avant lecture/écriture).

casser des exemples n'est pas un pb c'est facile à fixer, la question, c'est est-ce qu'on le veut, et si on le veut pourquoi ne pas avoir du code générique pour n'importe quel type?

thomas-lemaire commented 7 years ago

sans trop savoir où c'est utilisé, en voyant le code je me dis que l'intention est :

unsigned char pour un "petit int" (genre image de label)
char pour stocker un caractère

mais je ne sais pas où on utilise un vector