<div dir="auto"><div>[ Sorry about HTML crud, on mobile right now ]<br><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, Jan 22, 2020, 09:52 Christophe Leroy <<a href="mailto:christophe.leroy@c-s.fr">christophe.leroy@c-s.fr</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br>
-       struct linux_dirent __user * dirent;<br>
+       struct linux_dirent __user * dirent, *dirent0;</blockquote></div></div><div dir="auto"><br></div><div dir="auto">Patch looks better, but those names are horrid.</div><div dir="auto"><br></div><div dir="auto">Please just use "prev" for the previous dirent, and leave the current one as "dirent".</div><div dir="auto"><br></div><div dir="auto">I think that's going to be a lot more legible. But no numbering.</div><div dir="auto"><br></div><div dir="auto">      Linus</div></div>