<div dir="ltr"><br><br><div class="gmail_quote">On Sat Feb 21 2015 at 4:23:16 PM Ben Hoyt <<a href="mailto:benhoyt@gmail.com">benhoyt@gmail.com</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">When merging some changes while working on scandir, I noticed a minor issue with this commit:<div><br></div><div><a href="https://hg.python.org/cpython/rev/4f6f4aa0d80f" target="_blank">https://hg.python.org/cpython/rev/4f6f4aa0d80f</a><div><br></div><div>The definition of "struct win32_stat" has been moved to fileutils.h and renamed to "struct _Py_stat_struct", which is fine -- however, the old "struct win32_stat" definition is still present (but unused) in posixmodule.c.</div></div><div><br></div><div>So I think the old "struct win32_stat { ... }" definition can simply be removed from posixmodule.c now.</div></div></blockquote><div><br></div><div>I don't think win32_stat is part of the stable ABI so as long as everything keeps working then I don't see why it needs to stick around.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div><br></div><div>Also, unrelated to this commit, I notice the _Py_attribute_data_to_stat function (was attribute_data_to_stat) can't fail and always returns 0, and all callers ignore its return value anyway. Can it be changed to return void?</div></div></blockquote><div><br></div><div>Don't see why not since it's a private API. </div></div></div>