Re: [hatari-devel] WinUAUE CPU -> show EA

[ Thread Index | Date Index | More lists.tuxfamily.org/hatari-devel Archives ]


> Am 28.11.2023 um 19:01 schrieb Andreas Grabher <andreas_g86@xxxxxxxxxx>:
> 
> 
>>> Am 28.11.2023 um 18:10 schrieb Nicolas Pomarède <npomarede@xxxxxxxxxxxx>:
>>> 
>>> Le 28/11/2023 à 18:00, Andreas Grabher a écrit :
>>> 
>>> It does not affect the debugging output patch, but syncing with WinUAE: Was this left out intentionally?
>>> https://github.com/tonioni/WinUAE/commit/dd69a5ca3b55ce6bf49b7569f1df3b62618d6760 <https://github.com/tonioni/WinUAE/commit/dd69a5ca3b55ce6bf49b7569f1df3b62618d6760>
>> 
>> Hi
>> 
>> This part is not used in Hatari, you have "#ifndef WINUAE_FOR_HATARI" before so I didn't update it.
>> but you're right, it would be better to include it anyway.
>> 
>> will fix this
>> 
>> Nicolas
> 
> It makes comparing to WinUAE easier. Thank you for adding it. 
> 
> Another minor suggestion:
> 68kDisass.c includes stMemory.h and tos.h. Both are unused and could be removed. 
> 
I just recognized they are in fact used, but only if HAVE_CAPSTONE_M68K is true. So probably these headers could be included conditionally. 


Mail converted by MHonArc 2.6.19+ http://listengine.tuxfamily.org/