On 21 August 2017 at 14:33, Eric Engestrom <eric.engest...@imgtec.com> wrote: > On Monday, 2017-08-21 11:27:20 +0100, Emil Velikov wrote: >> From: Emil Velikov <emil.veli...@collabora.com> >> >> Flesh the duplicated functionality to a helper. >> Use const char * + sscanf to keep the compiler happy. > > Can you explain why you replaced strtol() with sscanf() in the commit > message? > > Reviewed-by: Eric Engestrom <eric.engest...@imgtec.com> > How about the following. I don't mind bringing strtol back if people prefer so.
"strtol discards the const qualifier of the endptr string. One could workaround it by using a local variable yet, sscanf seems like a shorter way to achieve the goal." Thanks Emil _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev
