NishanthMenon changed the topic of #openocd to: this is the place to discuss all things OpenOCD | Logs: https://libera.irclog.whitequark.org/openocd/
boru` has joined #openocd
boru has quit [Killed (NickServ (GHOST command used by boru`))]
boru` is now known as boru
tsal has quit [Ping timeout: 252 seconds]
tsal has joined #openocd
thinkfat has quit [Ping timeout: 252 seconds]
thinkfat has joined #openocd
cp- has joined #openocd
c4017__ has joined #openocd
c4017_ has quit [Ping timeout: 258 seconds]
akaWolf has quit [Ping timeout: 268 seconds]
akaWolf has joined #openocd
nerozero has joined #openocd
sbach has quit [Read error: Connection reset by peer]
sbach has joined #openocd
Haohmaru has joined #openocd
tarekb has joined #openocd
akaWolf has quit [Ping timeout: 252 seconds]
w00tSpeaks_ has joined #openocd
w00tSpeaks has quit [Quit: Konversation terminated!]
Guest32 has joined #openocd
Guest32 has quit [Client Quit]
shirleyquirk has joined #openocd
<shirleyquirk> just compiled openocd under cygwin-- got a couple `error: array subscript has type char[-Werror=char-subscripts]` errors
<shirleyquirk> solution was to replace `isspace(x)` and `isprint(x)` with `isspace((int)x)` etc.
<shirleyquirk> question is: how/where do i submit a pull request/bug report?
* Fleck pokes PaulFertser
<karlp> man that's a shitty warning.
<karlp> you're allowed to use signed ints as array indices
<karlp> but not _potentially_ signed chars,
<karlp> as it might "surprise" you o learn that you have a signed char
<shirleyquirk> it is a shitty warning, and even shittier that cygwin `ctypes` go out of their way to make it occur.
<shirleyquirk> why do i need to cast my char to an int before i'm allowed to know if it's a space or not
<karlp> fuck that warning entirely.
<karlp> looks like a grandfathered warning though, it's all the way back in gcc 4.1
<karlp> so now it's kept because "we must get that warning, even if it's fucking dumb"
<Haohmaru> why would you use char for an index tho
<karlp> why not?
<karlp> why does it matter?
<Haohmaru> it's meant for characters ;P~
<karlp> liek... "isspace()" ?
<karlp> apparenrlt some backends for those functions use array indexing, so passing a char, to the "char test functions" fails...
<Haohmaru> no idea what that is
<karlp> what does it look like they do?
<Haohmaru> i see.. it's C..
<Haohmaru> that explains it all ;]
<karlp> they formally take an int, even though you'd normally have a char you want to test
<karlp> like, tha man page even says it must be an unsigned char :)
<Haohmaru> "char" is special, whether it's signed or not was iirc merky
akaWolf has joined #openocd
<Haohmaru> not sure about C tho
<shirleyquirk> @Haohmaru you'd use a char for an index if you, say, `#define isspace(x) __lookup_table[x]`
<Haohmaru> how big would you make that LUT tho? ;P~
<shirleyquirk> 256 ints long
<shirleyquirk> so, i've made a bug report on sourceforge, but i can't see how to submit a pr, is that possible?
<tarekb> shirleyquirk: yes it's possible, please HACKING file
<shirleyquirk> cheers!
<tarekb> it's well formatted ;)
troth has quit [Quit: Leaving.]
<karlp> so, should we add -Wnochar-subscripts ? :)
<karlp> -Wno-char-subscripts sorry
<Haohmaru> x_x
shirleyquirk57 has joined #openocd
shirleyquirk57 has quit [Client Quit]
olerem has quit [Ping timeout: 240 seconds]
<shirleyquirk> patch submitted, i didn't change every use, e.g. in riscv.c there're some in parse_ranges that call isspace(char). sqlite defines their own, jimctl calls `issspace(UCHAR(x))` consistently.
shirleyquirk has quit [Quit: Client closed]
olerem has joined #openocd
emeb has joined #openocd
tlwoerner has quit [Remote host closed the connection]
crabbedhaloablut has quit [Remote host closed the connection]
crabbedhaloablut has joined #openocd
tarekb has quit [Read error: Connection reset by peer]
rowanthorpe has joined #openocd
shirleyquirk has joined #openocd
shirleyquirk has quit [Ping timeout: 246 seconds]
borneoa_ has quit [Read error: Connection reset by peer]
Steffanx- has quit [Read error: Connection reset by peer]
NishanthMenon_ has quit [Read error: Connection reset by peer]
NishanthMenon_ has joined #openocd
borneoa_ has joined #openocd
Steffanx- has joined #openocd
Haohmaru has quit []
Hawk777 has joined #openocd
c4017w has quit [Read error: Connection reset by peer]
rowanthorpe has quit [Ping timeout: 248 seconds]
c4017w has joined #openocd
akaWolf has quit [Ping timeout: 250 seconds]
nerozero has quit [Ping timeout: 240 seconds]
tlwoerner has joined #openocd
akaWolf has joined #openocd
akaWolf has quit [Ping timeout: 268 seconds]
zjason` has joined #openocd
zjason has quit [Ping timeout: 268 seconds]
tarekb has joined #openocd
<tarekb> Hi All,
<tarekb> I have submitted this patch http://openocd.zylin.com/#/c/6427/ adding the possiblity to add log messagaes (error/warning/debug) in configuration scripts
<tarekb> please do not hesitate to review it ;)
tarekb has quit [Read error: Connection reset by peer]
emeb has quit [Quit: Leaving.]
crabbedhaloablut has quit [Ping timeout: 244 seconds]
crabbedhaloablut has joined #openocd