Hi Dan, On 8/22/22 11:45, Dan Carpenter wrote: > Hi Dmitry, [..] > "err" not set. It was supposed to be set to zero at the start. But > better to say "ret = i;" here maybe? > > Why is i unsigned? It leads to unsightly casts. Presumably some static > checker insists on this... :/ Thanks! Will be addressed/reworked in v2. Thank you, Dmitry