Re: [hatari-devel] Patch for bits

[Eero Tamminen <oak@xxxxxxxxxxxxxx>]

Hi,

On 14.4.2024 11.37, Thomas Huth wrote:
> > > At first, I wanted to rename all "bits" variables in Hatari, but there're quite a lot over the files and also I don't see why  code in Hatari (excluding cpu core) should be dependant of some variables names that are used only for the cpu core, so I'd rather disable these enum when outside of the cpu core.
> > > That's a little hack, but I think it's better than renaming everything and adding in our docs that "bits" is now some kind of reserved word that can't be used anymore to name a variable.
> > >
> > > other method could be to split readcpu.h in 2 files and only include the file that don't declare the enum's in hatari's files, but that would create more differences with WinUAE's cpu core, which doesn't make things easier to keep track of changes in WinUAE.
> > >    
> > I don’t see many uses of bits outside the CPU core. I think it would be much cleaner to just rename them or leave it as is. It is just a warning (I think it is enabled by -Wshadow=local flag) and there are no impacts on functionality anyway. Adding compile-time checks that resolve to true of false depending on where a file is included from can cause bugs that are very hard to find.
>
> IMHO the proper solution would be to rename the entries in the enum in
> readcpu.h and submit that patch to Toni. Having a definition named "bits"
> in a header that is included all over the place sounds very fragile to me
> otherwise.

Agree 100%, I was about to suggest the same myself!


	- Eero