Ben je prends une ligne au pif:
cmp.b d1,d2 ; Equal ?C'est cool j'avais vu que c'est un comparaison, le commentaire ne me sert a rien. ^^
Par contre si tu avais écrit un truc genre "Compares the current char with the separator" j'aurai compris sans aller chercher 3 lignes avant et après.
moveq.l #-1,d0 ; #item Pareil, c'est pas très explicite…
Je dois aller regarder l'instruction pour comprendre que tu fous un -1 dans d0 donc que en fait #item ça veut dire "item number" donc que en fait ce que tu fais c'est "Initialize the index at -1"…
De même tu sors des "else" de n'importe où, mais je vois pas de "if … (then)" dans tes commentaires, donc ça ajoute de la confusion. Un mot plus approprié serait "otherwise".
Etc.
Aussi faut pas te sentir obligé de foutre un commentaire à côté de chaque instruction, si tu sens que tu peux tout expliquer sur la ligne suivante ou précédente (de préférence la précédente) avec une seule phrase intelligible
En fait tes commentaires de code ressemblent un peu au texte qu'on mettrait dans un diagramme, à la place du code, mais pour moi il ne commentent pas le code, ils commentent l'instruction, du coup on est quand même obligé de se taper tout le code ASM pour comprendre ce que tu fais. Et lire le code ASM c'est chiant, mais comme la plupart des gens ici (je pense) je le fais aussi bien sans les commentaires.
Pour être plus exact j'ai du lire le topic en entier pour avoir une idée de ce que pouvait faire ta routine parce que le commentaire en en-tête m'a laissé dans cet état:
Et au final j'ai quand même du aller fourrer ma tête dans les instructions ASM. ^^
Enfin je reste sur ce que j'ai dis hein, la routine en elle-même n'a probablement pas grand chose d'optimisable. Et je ne te suggère absolument pas de la coder en C
