[haiku-commits] Re: haiku: hrev44875 - in src: system/ldscripts/arm system/runtime_loader/arch/arm kits/debug/arch/arm .

  • From: "Ithamar R. Adema" <ithamar.adema@xxxxxxxxxxxxxxxx>
  • To: haiku-commits@xxxxxxxxxxxxx
  • Date: Thu, 22 Nov 2012 20:43:47 +0100

Hi Axel,

On 11/22/2012 08:33 PM, Axel DÃrfler wrote:
However, why would you want to set it to 1 for ARM? Even if we currently (or in the foreseeable future) do not support SMP, there is no real reason to restrict that one, especially because changing it is a binary compatibility issue.
I did not set it to that, it turned out to be set to that before I got involved. I agree this should be changed, but hey, it caught this error, and the results would even be worse if the constant ended up being more then the number of colors listed here (as the colors would be uninitialized for those, and the code would still try to use them).

I'm currently doing a sweep over the sources to find issues and get an overview of the amount of work still to do to get ARM into shape, I'll put this one on the TODO list to checkout once I start fixing up things structurally.

Regards,

Ithamar.


Other related posts: