Re: [hatari-devel] Videl Emu and TimerB

[Nicolas Pomarède <npomarede@xxxxxxxxxxxx>]

Le 08/03/2012 00:08, Laurent Sallafranque a écrit :
> Sorry for the empty mail.
>
> Maybe I could just change the last else
>
> /* Is it event count mode or not? */
> if (MFP_TBCR != 8)
> {
> /* Not event count mode, so handle as normal timer
> * and store result in 'MFP_TB_MAINCOUNTER' */
> MFP_ReadTimerB(false);
> }
> else if (bUseVDIRes)
> {
> /* HBLs are disabled in VDI mode, but TOS expects to read a 1. */
> MFP_TB_MAINCOUNTER = 1;
> }
> /* Special case when reading $fffa21, we need to test if the current
> read instruction */
> /* overlaps the horizontal video position where $fffa21 is changed */
> else <------------------- HERE
> {
> int FrameCycles, HblCounterVideo;
> int pos_start , pos_read;
>
>
> by else if (ConfigureParams.System.nMachineType != MACHINE_FALCON ) {
>
>
> Do you agree ?

Yes, the fist 2 'if' are needed ; you should add the falcon case between 
'else if (bUseVDIRes)' and 'else' (on your HERE mark)

Nicolas