[PC-DOS] HEROQUEST by Gremlin (1991) REMASTERED, - Modding, Bugfixing e Traduzione ITA/ESP devlog Opzioni

[raul]

Non ancora, ti ho dato la soluzione perché la volevi testare ma vanno ovviamente aggiunte tutte le descrizioni, non l'ho ancora fatto sul mio italian.bin, tu ovviamente dovrai fartelo sul tuo spanish.bin.

La versione italiana di oggi va bene, ma quella di ieri non cancellava i testi. Sembra incredibile!
Mi sono appena accorto che in fondo alle prime due compaiono solo le caratteristiche dell'arma. Non hai cambiato il puntatore sugli altri?

Stavo ragionando se spostare il prezzo sulla sinistra togliendo la frase e mettendo i due costi ad esempio:

L'ARMERIA
VENDE A 150
ACQUISTA A 75


COSTI ARMERIA
VENDITA: 150
ACQUISTO: 75

Sì, per favore, è una grande idea. Questo comincia a sembrare un negozio web di un supermercato.

Per recuperare spazio nel file language.bin ti consiglio di compattare tutte e sei queste voci:

Ho intenzione di occupare i byte liberi alla fine e di risalire la china.

Messaggio modificato da raul il 12 Apr 2024, 15:07

[TheRuler]

La versione italiana di oggi va bene, ma quella di ieri non cancellava i testi. Sembra incredibile!
Mi sono appena accorto che in fondo alle prime due compaiono solo le caratteristiche dell'arma. Non hai cambiato il puntatore sugli altri?

non è incredibile, non aggiorno l'archivio ogni 2 minuti, ci metto solo le versioni utili per un betatest, non quelle in lavorazione.
Ora ho terminato l'inventario, ma come ti ho già detto, quello è un lavoro che dovrai fare tu in spanish.bin

Stavo ragionando se spostare il prezzo sulla sinistra togliendo la frase e mettendo i due costi ad esempio:

L'ARMERIA
VENDE A 150
ACQUISTA A 75


COSTI ARMERIA
VENDITA: 150
ACQUISTO: 75

Per questa modifica serve liberare altro spazio, io sono arrivato in fondo!!!
 Screenshot_2024_04_12_165903.png ( 13.43k ) Numero di download: 0

Quale è meglio delle due versioni secondo te?

Per fortuna c'è TANTO alttro testo che si può eliminare che non serve a nulla.

Ho intenzione di occupare i byte liberi alla fine e di risalire la china.

Sei inguaribile. NON SERVONO A NULLA, non le vedrai mai! ;-)

Nel frattempo ho trovato altre stringhe che si possono unire:
 Screenshot_2024_04_12_164859.png ( 8.68k ) Numero di download: 0

Tutte quelle "ABCDEFGHIJKLMNOP" possono essere compattate in una sola. Ogni puntatore lo devi far puntare alla prima, stai attento che il puntatore di ogni stringa è duplicato anche in quella stringa strana senza testo (sono le coordinate degli slot di salvataggio) fatta così:
.......3.....3.....3.....3.....3.....3.....3.....3....q5....
EDIT: Errore, non si possono compattare. vanno lasciate così perché ogni volta che carichi o salvi, il gioco copia lì il gruppo di nomi degli slot. Se compatti tutto in una riga, funziona lo stesso ma tutti gli slot avranno lo stesso nome di quello del primo slot.

ci sono anche altre stringhe che non ho mai visto in gioco, hai idea di cosa possano essere?

QUEST TREASURE.
NONE HELD.
ARMOUR WORN.
NO ARMOUR.
WEAPON IN USE.
POTIONS TAKEN.
NO POTION TAKEN....
LOAD WHICH DEFINITION....
SAVE WHICH DEFINITION

io le ho eliminate tutte

[raul]

non è incredibile, non aggiorno l'archivio ogni 2 minuti, ci metto solo le versioni utili per un betatest, non quelle in lavorazione.

Te l'avevo detto perché non sapevo se fosse un bug o se non l'avessi finito.

Quale è meglio delle due versioni secondo te?

Dovremmo valutare cosa è meglio dal punto di vista visivo e funzionale, ma in linea di principio mi piace di più questo:

L'ARMERIA
VENDE A 150
ACQUISTA A 75

ci sono anche altre stringhe che non ho mai visto in gioco, hai idea di cosa possano essere?

QUEST TREASURE.
NONE HELD.
ARMOUR WORN.
NO ARMOUR.
WEAPON IN USE.
POTIONS TAKEN.
NO POTION TAKEN....
LOAD WHICH DEFINITION....
SAVE WHICH DEFINITION

io le ho eliminate tutte

Fa ancora male, forse sono importanti e non vengono attivati.

Messaggio modificato da raul il 12 Apr 2024, 17:16

[TheRuler]

Dovremmo valutare cosa è meglio dal punto di vista visivo e funzionale, ma in linea di principio mi piace di più questo:

L'ARMERIA
VENDE A 150
ACQUISTA A 75

Ovviamente va le cifre le farei comparire solo quando passi il mouse sull'arma.
Con il mouse non sull'arma si leggerebbe:
L'ARMERIA
VENDE A
COMPRA A

Fa ancora male, forse sono importanti e non vengono attivati.

Non esiste NESSUN riferimento nel codice, dunque sono stringhe orfane.
Potrei aggiungere a mno il codice ma non vedo la necessità o il vantaggio.
Vuoi che quando clicchi su un'armatura compaia un messaggio con scritto "armatura indossata"? non credo.
Devono far parte di un set di azioni non utilizzate, sono presenti anche nel gioco Amiga.

EDIT: trovate. Sono della versione amiga.
 Screenshot_2024_04_12_190642.png ( 596.24k ) Numero di download: 1


La cosa aggiunge un nuovo tassello alla storia della programmazione di quest gioco. Essendo le versioni uscite nello stesso anno, è facile supporre che la scrittura del testo l'avessero affidata ad un tem trasversale ed utilizzato solo le parti necessarie per ogni gioco.

[raul]

Ovviamente va le cifre le farei comparire solo quando passi il mouse sull'arma.
Con il mouse non sull'arma si leggerebbe:
L'ARMERIA
VENDE A
COMPRA A

Pensavo che sarebbe rimasto con 0000.
Allora è meglio l'altro modo.

Non esiste NESSUN riferimento nel codice, dunque sono stringhe orfane.
Potrei aggiungere a mno il codice ma non vedo la necessità o il vantaggio.
Vuoi che quando clicchi su un'armatura compaia un messaggio con scritto "armatura indossata"? non credo.
Devono far parte di un set di azioni non utilizzate, sono presenti anche nel gioco Amiga.

Cosa avevano in mente? Che l'attrezzatura si sarebbe usurata e che avrebbero dovuto ricomprarla? Forse ne saprete di più in seguito.

[TheRuler]

Pensavo che sarebbe rimasto con 0000.
Allora è meglio l'altro modo.

Sto provando a rivedere il codice di cancellazione perché ha uno strano comportamento, se provi a cliccare diverse volte sulla parte vuota a sinistra cattura il mouse e lo rilascia solo dopo che hai cliccato una seconda volta. Forse l'allargamento dell'area che gli ho fatto è troppo "pesante".

Cosa avevano in mente? Che l'attrezzatura si sarebbe usurata e che avrebbero dovuto ricomprarla? Forse ne saprete di più in seguito.

Avevano in mente di usare la parte testuale per la versione Tandy, che non aveva abbastanza ram per la grafica.
Worn è "indossato", non "logorato" in questo caso.
Quel testo non è usato nella versione DOS VGA, né in quella EGA né in quella TANDY. Fidati se ti dico che è stato lasciato lì dalla versione Amiga.

[raul]

Sto provando a rivedere il codice di cancellazione perché ha uno strano comportamento, se provi a cliccare diverse volte sulla parte vuota a sinistra cattura il mouse e lo rilascia solo dopo che hai cliccato una seconda volta. Forse l'allargamento dell'area che gli ho fatto è troppo "pesante".

Spero che tu riesca a trovare una soluzione.

[TheRuler]

Sto provando a rivedere il codice di cancellazione perché ha uno strano comportamento, se provi a cliccare diverse volte sulla parte vuota a sinistra cattura il mouse e lo rilascia solo dopo che hai cliccato una seconda volta. Forse l'allargamento dell'area che gli ho fatto è troppo "pesante".

Spero che tu riesca a trovare una soluzione.

Ho ridotto l'area di cancellazione e il problema mi sembra scomparso.

 dopo2.jpg ( 275.75k ) Numero di download: 1


offset 0x08666 --> 9200BFC838BD50

[raul]

Domani rimuoverò la modifica precedente e metterò questa nuova.
Buon lavoro, come sempre.

Messaggio modificato da raul il 14 Apr 2024, 22:19

[TheRuler]

Questa nuova funzione è stata una vera sfida. Il pezzo di codice più complesso che mi sia trovato a fare in assembly da quando ho cominciato questo progetto.
Mi ero accorto di aver fatto lo stesso errore dei programmatori (nella missione dell'equipaggiamento rubato) quando ho scritto la funzione di vendita degli oggetti all'armeria: mi sono dimenticato di rimuovere l'oggetto venduto dallo stato di "equipaggiato"!
La conseguenza era che fino a quando il giocatore non cambiava l'equipaggiamento, se l'oggetto venduto fosse stato "in uso" avrebbe continuato ad esserlo anche se non più presente.

Nel metter a posto l'errore mi sono messo a pensare come sarebbe stato bello poter passare gli oggetti tra gli eroi, ma mancando una interfaccia grafica non era possibile. L'unica cosa che abbiamo a disposizione sono glli scroll... SI PUO FARE!
Il problema è sempre lo spazio. Il codice sarà sicuramente enorme, e dove lo metto?!!
Ho confrontato circa 20 memory dump di 3 mesi e ho trovato un'area sufficientemente grande mai sovrascritta! *** va testata a fondo in partite lunghe senza mai uscire ***
Mi sono messo subito a fare prove di fattibilità e stasera ho finalmente ripulito il codice e integrato nell'exe.


Siete pronti?

 Screenshot_2024_04_16_210211.png ( 92.27k ) Numero di download: 0


 Screenshot_2024_04_16_210944.png ( 98.71k ) Numero di download: 0


La cosa davvero incredibile è che FUNZIONA. Davvero ancora non mi capacito.


Ovviamente ho dovuto aggiungere le stringhe nuove nel file ITALIAN.BIN. Raul, verifica come l'ho fatto confrontando l'archivio al solito posto, questa volta è un filino complicato, perché ho dovuto aggiungere 5 opzioni.

Soluzione 2 corretta e ottimizzata qui.

[raul]

[raul]

Non credo di dover commentare che questa nuova funzione è fantastica e non voglio che le mie parole suonino come un'ingrata, ma non sarebbe meglio risparmiare quei byte liberi per altre funzioni come i movimenti dei nemici?

[TheRuler]

Non credo di dover commentare che questa nuova funzione è fantastica e non voglio che le mie parole suonino come un'ingrata, ma non sarebbe meglio risparmiare quei byte liberi per altre funzioni come i movimenti dei nemici?

Per quello non è un problema, se mai qualcuno troverà il modo (non io, non sono in grado, ci ho già sprecato due settimane e non ci ho cavato fuori nulla di buono) avrà tutto lo spazio che vuole, perché il codice del movimento andrebbe SOSTITUITO.

[raul]

Potrebbe anche essere necessario un numero maggiore di byte, ma se voi dite che va bene, così sia. Santo cielo, il mio lavoro si sta accumulando. XD

Sei riuscito a fare qualcosa per la compravendita dell'armeria?

[TheRuler]

il mio lavoro si sta accumulando. XD

immagino che dovrò darti una mano, ho modificato pesantemente il file language.bin.
Tieni presente questa pagina, perché anche le opzioni aggiunte hanno un puntatore assoluto.

Sei riuscito a fare qualcosa per la compravendita dell'armeria?

temo di non capire cosa chiedi. La funzione sopra E' ESATTAMENTE la compravendita + scambio nell'armeria.

[raul]

Questo è ciò di cui sto parlando.

Stavo ragionando se spostare il prezzo sulla sinistra togliendo la frase e mettendo i due costi ad esempio:

L'ARMERIA
VENDE A 150
ACQUISTA A 75


COSTI ARMERIA
VENDITA: 150
ACQUISTO: 75

Messaggio modificato da raul il 16 Apr 2024, 21:44

[raul]

il mio lavoro si sta accumulando. XD

immagino che dovrò darti una mano, ho modificato pesantemente il file language.bin.

Non ho potuto fare nulla per un po', ma appena potrò, lavorerò lentamente sugli arretrati. Se avrò bisogno di aiuto, ve lo farò sapere, ma non preoccupatevi, non è importante, la priorità è andare avanti con questo progetto.

[TheRuler]

L'ARMERIA
VENDE A 150
ACQUISTA A 75


COSTI ARMERIA
VENDITA: 150
ACQUISTO: 75

ho dovuto ridurre l'area di cancellazione per il problema del click catturato, ricordi?
Andare a cancellare anche in quella zona "costa" troppo, ad ogni modo non credo serva più vista la nuova funzione.

[raul]

Sì, me lo ricordo, ma non sapevo che influenzasse questa nuova funzione. È un peccato, mi piaceva molto l'idea, dava dinamismo all'armeria.

Che armeria che state creando ehhhh, se questo gioco fosse della ubisoft, farebbero un sacco di soldi con le vendite del pagamento reale.

Messaggio modificato da raul il 16 Apr 2024, 22:41

[TheRuler]

Mi sono accorto di un bug nella magia Genio. Era davvero bacata:
Se spingo Annulla o tasto destro lo considera come RIVELARE.

Per fortuna si è trattato solo di invertire la logica:

SOLUZIONE:
at offset 0x9AC0 change 027513 to 00741D
at offset 0x9AC7 change FE75 to 0174

Sì, me lo ricordo, ma non sapevo che influenzasse questa nuova funzione. È un peccato, mi piaceva molto l'idea, dava dinamismo all'armeria.

si può sempre fare, basta rinunciare a voler mettere a posto e convivere con il problema della cattura del click.
Secondo me però l'armeria è già abbastanza piena.

[raul]

Mi sono accorto di un bug nella magia Genio. Era davvero bacata:
Se spingo Annulla o tasto destro lo considera come RIVELARE.

Giusto, ho dimenticato di dirlo in quel momento.
Grazie.

si può sempre fare, basta rinunciare a voler mettere a posto e convivere con il problema della cattura del click.

Lascia perdere, preferisco che l'armeria sia impeccabile. È un peccato perché mi sarebbe piaciuto mettere i veri nomi delle armature come sono nel gioco da tavolo.

Messaggio modificato da raul il 16 Apr 2024, 23:16

[TheRuler]

So di poter ridurre tanto il codice che ho creato e ottimizzarlo. Ho in mente una logica che evita tutti quei compare e test e li riduce ad uno solo, sfruttando la scelta dell'eroe che genera già il valore 00,01,02,03 in [AB7A] e il fatto che le locazioni di memoria dove l'oggetto viene salvato sono a distanza 1A tra loro. Devo solo studiarmi la moltiplicazione in assembly.
Poi devo vedere se è possibile implementare la stessa funzione anche in-game, per passare anche gli oggetti speciali. unico problema non esiste un controllo di distanza eroe-eroe, solo eroe-mostro, dunque sarebbe possibile passarsi ogggetti anche non a contatto. Secondo me un problema minore, se si potesse passare l'oggetto speciale.

mi sarebbe piaciuto mettere i veri nomi delle armature come sono nel gioco da tavolo.

Cosa intend? hanno già gli stessi nomi.

Per i prezzi, e se mettessimo il doppio valore da qualche parte dove già si cancella?
devo vedere se c'è lo spazio fisico, visto che "ARMATURA PESANTE" è il nome più lungo.
magari sopra a quello che già c'è, qualche esempio che ho in mente:

/75
150

+75
150

150/75

150(75)

1
5
0

/

7
5

 Screenshot_2024_04_12_115415.jpg ( 145.49k ) Numero di download: 0



metterlo a destra non comporta cambio finestra, metterlo sotto a sinistra (che sarebbe più bello in effetti) vorrebbe dire aumentare la finestra di cancellazione e acuire il problema del click catturato.

[TheRuler]

Ho ottimizzato il precedente codice di vendita e scambio riducendolo da 144 byte a 92, utilizzando un trucco interessante per la parte di scambio:
invece di controllare le 4 risposte individualmente e settare 4 indirizzi di memoria diretti, siccome la distanza relativa tra di loro è sempre 1A, e le risposte possono essere 00,01,02,03, è bastata una operazione di moltiplicazione tra la risposta e 1A per creare un indirizzo univoco per eroe relativo al primo indirizzo (barbaro), e dunque aggiungerlo all'indirizzo del barbaro.

CODICE

START OF THE NEW CODE:

1EF:81F1  56                  push si
1EF:81F2  55                  push bp
1EF:81F3  51                  push cx
1EF:81F4  52                  push dx
1EF:81F5  E9F246              jmp  0000C8EA ($+46f2)      (down)    <-- JUMPS TO THE NEW LOCATION
1EF:81F8  E8F206              call 000088ED ($+6f2)
1EF:81FB  EBDF                jmp  short 000081DC ($-21)  (up)


1ef:C8EA    BB 0E 00        mov bx, 0xe                            <-- "SELL/GIVE TO" STRING
1ef:C8ED    E8 FD BF        call 0x88ed                            <-- DRAW THE SCROLL
1ef:C8F0    5A                pop dx
1ef:C8F1    59                pop cx
1ef:C8F2    5D                pop bp
1ef:C8F3    5E                pop si
1ef:C8F4    80 3E 7A AB 00    cmp byte ptr [0xab7a], 0            <-- IF SELL
1ef:C8F9    75 07            jne 0xc902
1ef:C8FB    D1 ED            shr bp, 1                            <-- HALVE THE PRICE
1ef:C8FD    01 6C 09        add word ptr [si + 9], bp            <-- AND ADD THE AMOUNT TO STASH
1ef:C900    EB 2B            jmp 0xc92d
1ef:C902    80 3E 7A AB 01    cmp byte ptr [0xab7a], 1            <-- IF GIVE
1ef:C907    75 31            jne 0xc93a
1ef:C909    BB 2E 00        mov bx, 0x2e                        <-- "CHOSE HERO" STRING
1ef:C90C    56                push si
1ef:C90D    55                push bp
1ef:C90E    51                push cx
1ef:C90F    52                push dx
1ef:C910    E8 DA BF        call 0x88ed                            <-- DRAW THE SCROLL
1ef:C913    5A                pop dx
1ef:C914    59                pop cx
1ef:C915    5D                pop bp
1ef:C916    5E                pop si
1ef:C917    80 3E 7A AB 04    cmp byte ptr [0xab7a], 4            <-- CHECK THE CHOSEN HERO
1ef:C91C    73 1C            jae 0xc93a
1ef:C91E    6B 1E 7A AB 1A    imul bx, word ptr [0xab7a], 0x1a    <-- CREATE A RELATIVE ADDRESS BASED ON CHOSEN HERO
1ef:C923    81 C3 71 46        add bx, 0x4671
1ef:C927    85 0F            test byte ptr [bx], cx                <-- IF ALREADY OWNED BY THE RECEIVER
1ef:C929    75 12            jne 0xc93d
1ef:C92B    09 0F            or byte ptr [bx], cx                <-- ADD IT TO RECEIVER'S INVENTORY
1ef:C92D    09 CA            or dx, cx
1ef:C92F    F7 D1            not cx                                <-- INVERT THE EQUIPMENT TO BE ABLE TO "AND" IT
1ef:C931    21 4C 0E        and word ptr [si + 0xe], cx            <-- REMOVE ITEM FROM GIVER'S INVENTORY
1ef:C934    20 4C 11        and byte ptr [si + 0x11], cl        <-- UNEQUIP GIVER'S WEAPON
1ef:C937    20 6C 13        and byte ptr [si + 0x13], ch        <-- UNEQUIP GIVER'S ARMOUR
1ef:C93A    E9 BE B8        jmp 0x81fb
1ef:C93D    BB 00 00        mov bx, 0                            <-- "ALREADY OWNED" STRING
1ef:C940    E9 B5 B8        jmp 0x81f8                            <-- BACK TO PRIMARY FUNCTION

SOLUTION SELL/GIVE in the ARMORY:
0x8611 --> 56555152E9F246E8F206EBDF
BB0E00E8FDBF5A595D5E803E7AAB007507D1ED016C09EB2B803E7AAB017531BB2E0056555152E8DA
BF5A595D5E803E7AAB04731C6B1E7AAB1A81C37146850F7512090F09CAF7D1214C0E204C11206C13E
9BEB8BB0000E9B5B8909090909090

dal punto di vista del giocatore non è cambiato nulla, a parte il check di uscita (con annulla o tasto dx) del secondo scroll che non funzionava , ma potevo evitare di implementare una soluzione più elegante?

EDIT: modificata la logica di rimozione dell'equipaggiamento indossato. Non avevo tenuto conto che si potesse vendere anche qualcosa di non indossato. Grazie a raul che se ne era accorto qui.


per quanto riguarda il prezzo di vendita avevo pensato di metterlo qui, che è un ottimo compromesso tra estetica e limiti tecnici (sia per la cancellazione schermo che per le dimensioni del language.bin)
 Screenshot_2024_04_18_085549.png ( 117.78k ) Numero di download: 1

[TheRuler]

Ho allargato nuovamente l'area perché l'idea di spostare il prezzo in basso a sinistra continuava ad assillarmi.

Questa è la nuova area di cancellazione:
 Immagine_2024_04_18_161528.jpg ( 221.95k ) Numero di download: 5


E questa la nuova armeria:
 Screenshot_2024_04_18_161441.png ( 117.63k ) Numero di download: 2

SOLUZIONE 4
at offset 0x08666 change 0800BF9046BD90 to 9000BF203CBD84

Il problema della cattura del click c'è ancora anche se meno probabile. Cercherò di risolverlo mantenendo questa estetica perché mi piace molto.

[raul]

WOWWWWWWWWWWWWWWW
Dovrò leggerli attentamente, non so quale usare.

Il problema della cattura del click c'è ancora anche se meno probabile. Cercherò di risolverlo mantenendo questa estetica perché mi piace molto.

Sì, per favore, in modo da poter espandere il testo degli oggetti al nome originale del gioco da tavolo. Penso anche che si possano recuperare i byte liberi per il file .bin, no?


Messaggio modificato da raul il 18 Apr 2024, 16:35