Re: [hatari-devel] Mega ST(E)

[Nicolas Pomarède <npomarede@xxxxxxxxxxxx>]

Le 28/05/2016 à 19:49, Eero Tamminen a écrit :
> Hi,
>
> On 05/28/2016 08:05 PM, Thomas Huth wrote:
> > FYI, I've now committed a change that allows to select "Mega ST" and
> > "Mega STE" machine types now. The RTC option is gone - just like with
> > the real machines, the RTC is only available with the Mega ST/Mega STE
> > machine type now.
>
> I noticed that in "if"s in:
> - clocks_timings.c
> - ioMem.c
> - video.c
>
> You didn't use the isMachine*() inline functions
> you had added in the previous commit.  Was there
> some reason for this?

And maybe add some isMachineTT and isMachineFalcon ? Even if this just 
compare to one machine type, it would make the machine comparison more 
consistent in all case and completely abstract the
"ConfigureParams.System.nMachineType" part in the rest of the sources ?

Nicolas