Hi Praveen, On Fri, 2021-10-29 at 21:11 +0530, Praveen Kumar wrote: > However, as mentioned in previous patch, we can also make *i* as > unsigned short and that should IMO remove the requirement of cast. I agree, it should remove the cast but I think I'll leave `idx` as it is. > But again, this works as well. I'm fine with either. Thanks and sorry > for the confusion. No worries, thanks for looking at my changes. It's good to double check everything :) Thanks, Karolina
