FrOSt-Foundation / FrOSt

Dépôt officiel de FrOSt - OS communautaire Français pour 0x10c
GNU General Public License v3.0
13 stars 5 forks source link

Printf ne fonctionne plus #17

Closed azertyfun closed 11 years ago

azertyfun commented 11 years ago

La détection et l'initialisation des périphériques fonctionne très bien (voir 3f3f7f77a0), mais la console ne marche plus : au premier retour à la ligne, POUF, plus rien. J'ai l'impression que printf a du mal avec les retours à la ligne, je vais tenter de voir pourquoi. :(

azertyfun commented 11 years ago

Je dois bien avouer que je ne trouve pas d'où vient l'erreur, quelqu'un a une idée ?

Yamakaky commented 11 years ago

Euh, il est où ? XD

azertyfun commented 11 years ago

dans DriverEcran.dasm ^^

Yamakaky commented 11 years ago

Bon bah je vais voir ça.

C'est trop grisant ces derniers jours, on arrête pas de switcher entre 50 pages ^^ Comparé au fofo, cest plus fun !

azertyfun commented 11 years ago

T'as pas trouvé ? :( Et Faërie, ptêt qu'il va trouver l'erreur à la con lui ^^

Yamakaky commented 11 years ago

j'ai pas vraiment cherché '^^ Je vois ça demain, promis. Voir ce soir

azertyfun commented 11 years ago

Bon ben j'ai comparé entre la dernière version et un commit plus ancien, y'a vraiment rien qui change si ce n'est les labels (qui sont corrects évidemment ^^). Donc le problème ne vient pas de la. Je vais chercher d'où ça vient mais je vois pas d'où du coup vu que ce n'est que au retour à la ligne que ça merde.

azertyfun commented 11 years ago

Okay, donc apparemment c'est pas là que ça merde, c'est juste que le printf, ainsi que le printc, refusent d'afficher des caractères plus loin que la première ligne de l'écran. Pourtant, puts fonctionne parfaitement. Étrange. EDIT : WAIT là printf marche, je teste un truc 2 sec. EDIT2 : Bon en fait j'ai l'impression que le printf bug juste à l'intérieur de la console. :/ EDIT3 : Finalement c'est bien le retour à la ligne qui passe pas ^^'

L3nn0x commented 11 years ago

Je regarde et je vous dis.

azertyfun commented 11 years ago

Merci parce que là je sèche ^^

L3nn0x commented 11 years ago

J'ai trouvé : dans p_return, il faut ajouter add I,[écran_actif] à la fin avant de quitter la fonction.

azertyfun commented 11 years ago

ALLELUJA merci. Quelqu'un pou commit ? Je suis sur mon portable là :p

L3nn0x commented 11 years ago

Moi aussi, mais dans 5 minutes je pourrais.

azertyfun commented 11 years ago

Du coup si la console fonctionne bien tu peux merger avec indev.

L3nn0x commented 11 years ago

C'est fait.

azertyfun commented 11 years ago

Euh chez moi je rajoute la ligne (parce que tu l'as commit dans indev et pas dans update-hw), mais ça ne marche pas pour autant. Je te ferais remarquer qu'il faut tester sur la branche hw, parce que avec ou sans ta ligne de code le printf fonctionnait bien en indev ^^'

L3nn0x commented 11 years ago

Ah, mince, j'avais pas vu qu'il y avait une nouvelle branche... Et t'as raison, j'avais mal lu sur mon portable, c'était bon.

azertyfun commented 11 years ago

Quelqu'un essaie en remplacant les defines de printf pqr les nbres originaux ? Sinon je fait ça demain ^^

azertyfun commented 11 years ago

ILLUMINATION ! Faut pas mettre les crochets à S[CREEN_WIDTH] ET [SCREEN_HEIGHT] !! Dire que j'ai planché une heure là-dessus xD

L3nn0x commented 11 years ago

C'est pas ma faute ! :angel:

azertyfun commented 11 years ago

Nan c'est la mienne j'ai fait un bête chercher/remplacer sans réfléchir xD Je corrige toossa demain :)

Yamakaky commented 11 years ago

La branche update-hw est pas destinée à être éternelle, juste à updater le hw ^^ C'est pas une autre indev, faut pas hésiter à en créer d'autres. Regardez dans le readme, j'ai mis le lien d'un article pas mal sur la gestion des branch.

azertyfun commented 11 years ago

Pas con pas con. Bon demain je debug et si la console remarche je merge la branche dans indev. Je dirais même que après on pourrait merger indev avec master pisque on serait totalement compatibles avec devCPU :p

L3nn0x commented 11 years ago

Je peux m'occuper de corriger ces erreurs.

azertyfun commented 11 years ago

Bon ben ok, et dans la bonne vranche ce coup-ci :p

Yamakaky commented 11 years ago

L'OS est prêt pour une 0.1 ? (ou .2 plutôt je crois) Si non, on ne merge pas l'indev dans master. cf README XD

L3nn0x commented 11 years ago

Oui, oui...

L3nn0x commented 11 years ago

Voilà.

azertyfun commented 11 years ago

Corrigé, je ferme. Par contre, l'interruption 0x4000 ne marche pas :( (Voir issue #19)