Rockbox mail archive
Subject: Re: cvs: apps settings.c,1.159,1.160Re: cvs: apps settings.c,1.159,1.160
- Contemporary messages sorted: [ by date ] [ by thread ] [ by subject ] [ by author ] [ by messages with attachments ]
From: Björn Stenberg <bjorn_at_haxx.se>
Date: Wed, 11 Jun 2003 14:13:24 +0200
Hardeep Sidhu wrote:
> if (((type==INT) && (*intvar != oldval)) ||
> - ((type==BOOL) && (*boolvar != oldval))) {
> + ((type==BOOL) && ((int)*boolvar != oldval))) {
> if (type==INT)
The bool variable can be located at an odd address. Trying to read an integer from an odd address will result in a CPUAdrErr exception.
It would probably be better to cast oldval to a bool instead.
Date: Wed, 11 Jun 2003 14:13:24 +0200
Hardeep Sidhu wrote:
> if (((type==INT) && (*intvar != oldval)) ||
> - ((type==BOOL) && (*boolvar != oldval))) {
> + ((type==BOOL) && ((int)*boolvar != oldval))) {
> if (type==INT)
The bool variable can be located at an odd address. Trying to read an integer from an odd address will result in a CPUAdrErr exception.
It would probably be better to cast oldval to a bool instead.
-- BjörnReceived on 2003-06-11
Page template was last modified "Tue Sep 7 00:00:02 2021" The Rockbox Crew -- Privacy Policy