Fix C-style cast warnings with clang-tidy

[masterleinad]

Running clang-tidy locally with our current settings, I still see some complaints about C-style casts.

[crtrott]

I don't really see the reason to have auto* there why not just auto:
https://godbolt.org/z/6Eo178jjx

Not blocking, I mean it might make it a bit clearer that we are looking at a const ptr, not a ptr of const. But still, if anyone else feels like get rid of * please feel free to say.

[dalg24]

I agree with Christian suggesting to use auto instead of auto* in general but I don't think it is worth arguing about it, especially for that specific code.

[masterleinad]

Note that there is another clang-tidy check for this and that the LLVM Coding Standards advise to make it obvious if an auto-typed variable is a pointer, see https://clang.llvm.org/extra/clang-tidy/checks/readability/qualified-auto.html.