10000 Ajout $512 (ERRNB stratsed) by jedeoric · Pull Request #2 · assinie/Hyper-Basic · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Ajout $512 (ERRNB stratsed) #2

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

jedeoric
Copy link
Contributor

No description provided.

@jedeoric jedeoric requested a review from assinie January 28, 2020 10:11
@assinie
Copy link
Owner
assinie commented Jan 28, 2020

Le problème est que j'ai déjà utilisé errnb en minuscules pour le numero d'erreur du basic (adresse $07B7).
Si on utilise un assembleur qui ne fait pas la différence MAJUSCULES/minuscules ça posera un problème (et on n'est pas non plus à l'abri d'une erreur)
Il faudrait donc que je renomme celle du basic pour éviter les pbs.

Il y a d'autres variables que j'ai ajouté dans le source et que je n'ai pas reportées dans le fichier info.

@jedeoric
Copy link
Contributor Author

Oui en fait, le ERRNB c'est celui qui est documenté dans le manuel du développeur telestrat ($512) et ce que j'avais vu c'était le ERRNB du stratsed. Effectivement, je pense qu'il faudrait renommé celui du basic qui est différent (je pense qu'il doit sortir les syntax error, illegal quantity error and co).

Je fais la modif ou tu la fais ? (cela ne me dérange pas, mais je risque de ne pas prendre la même norme que toi

@assinie
Copy link
Owner
assinie commented Jan 28, 2020

Je peux le faire, j'ai aussi d'autres corrections dans ce fichier qu'il faut que je transfère dans le dépôt.
Il faut juste que je lui trouve un nom qui tienne la route.

Le problème avec les variables des roms du Telestrat, c'est que ce n'est pas simple de s'y retrouver. Certaines sont définies dans un document, d'autres dans un autre, certaines sont décrites mais pas nommées et il y en a qui ne sont décrites nulle part.
Je n'ai pas trouvé de référentiel complet, j'utilise principalement le manuel du développeur et Telestrat Système m'était conté.

@jedeoric
Copy link
Contributor Author

il y a aussi le désassemblage de teleass avec les labels. Mais, oui, il n'y a pas une doc uniforme.

Sinon dans telestrat.inc de cc65 : https://github.com/cc65/cc65/blob/master/asminc/telestrat.inc

A la fin, il y a les adresses Stratsed

@assinie
Copy link
Owner
assinie commented Jan 28, 2020

Je vais refaire une passe sur ce fichier, en fait je ne m'en étais servi qu'au début et je n'ai pas reporté systématiquement les modifs faites dans le source Hyperbasic vers ce fichier.

Pour ERRNB, le truc c'est que c'est aussi le nom d'une instruction/variable Hyperbasic.
C'est pour ça que dans le source il y a ERRNB et errnb, un pour l'instruction et un pour la variable conrrespondate, et que je n'avais pas nommé l'adresse $0512

J'aurai bien fait un fichier avec les variables communes à toutes les cartouches (en gros celles de Telemon) et un fichier spécifique par cartouche, mais da65 n'a pas de fonction include...

@jedeoric
Copy link
Contributor Author

L'idéal aurait été de coller l'include de telestrat.inc mais oui, je ne sais pas comment faire avec da65

@assinie
Copy link
Owner
assinie commented Jan 28, 2020

Oui c'est sûr, c'est ce que j'ai prévu pour l'assemblage du fichier source mais pour le moment on ne peut pas ré-assembler le source directement à cause de certains labels qui ont été créés automatiquement et que je dois renommer.
J'attends de fnir l'identification de toutes les variables et routines Hyperbasic avant de faire les modifs pour que l'assemblage soit possible.

Je suis en train de revoir le fichier .info, je vais aussi m'en servir pour la cartouche Telematic que l'Hyperbasic peut appeler directement.

7598

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants
0