
<html>

<head>

<meta http-equiv="Content-Type" content="text/html; charset=windows-1256">

<meta name="Generator" content="Microsoft Exchange Server">

<!-- converted from text --><style><!-- .EmailQuote { margin-left: 1pt; padding-left: 4pt; border-left: #800000 2px solid; } --></style>

</head>

<body>

<div>

<div>

<div style="font-family:Calibri,sans-serif; font-size:11pt">I think posixmodule is a great candidate for splitting up by platform rather than function, as the whole file is packed with ifdef. It's really only lacking a volunteer to do it, but we could start

 here (ie. make posixmodule_nt.c for the Windows impl, etc.) and progressively move function implementations out over time? All the module setup and probably most of the Python layer can stay where it is.<br>

<br>

More likely we're going to get bogged down discussing it again though, so if that happens my vote is to just make posixmodule.c 800 lines longer.<br>

<br>

Cheers,<br>

Steve<br>

<br>

Top-posted from my Windows Phone</div>

</div>

<div dir="ltr">

<hr>

<span style="font-family:Calibri,sans-serif; font-size:11pt; font-weight:bold">From:

</span><span style="font-family:Calibri,sans-serif; font-size:11pt"><a href="mailto:solipsis@pitrou.net">Antoine Pitrou</a></span><br>

<span style="font-family:Calibri,sans-serif; font-size:11pt; font-weight:bold">Sent:

</span><span style="font-family:Calibri,sans-serif; font-size:11pt">ı2/ı13/ı2015 5:44</span><br>

<span style="font-family:Calibri,sans-serif; font-size:11pt; font-weight:bold">To:

</span><span style="font-family:Calibri,sans-serif; font-size:11pt"><a href="mailto:python-dev@python.org">python-dev@python.org</a></span><br>

<span style="font-family:Calibri,sans-serif; font-size:11pt; font-weight:bold">Subject:

</span><span style="font-family:Calibri,sans-serif; font-size:11pt">Re: [Python-Dev] PEP 471 (scandir): Poll to choose the implementation (full C or C+Python)</span><br>

<br>

</div>

</div>

<font size="2"><span style="font-size:10pt;">

<div class="PlainText">On Fri, 13 Feb 2015 08:35:00 -0500<br>

Ben Hoyt <benhoyt@gmail.com> wrote:<br>

> <br>

> If we go ahead with the all C approach, I'd be in favour of<br>

> refactoring a little and putting the new scandir code into a separate<br>

> C file. There are two ways to do this: a) sticking with a single<br>

> Python module and just referencing the non-static functions in<br>

> scandir.c from posixmodule.c, or b) sharing some functions but making<br>

> _scandir.c its own importable module. Option (a) is somewhat simpler<br>

> as there's not module setup stuff twice, but I don't know if there's a<br>

> precedent for that way of doing things.<br>

<br>

The _io module already does things that way (the (a) option, I mean).<br>

<br>

Regards<br>

<br>

Antoine.<br>

_______________________________________________<br>

Python-Dev mailing list<br>

Python-Dev@python.org<br>

<a href="https://mail.python.org/mailman/listinfo/python-dev">https://mail.python.org/mailman/listinfo/python-dev</a><br>

Unsubscribe: <a href="https://mail.python.org/mailman/options/python-dev/steve.dower%40microsoft.com">

https://mail.python.org/mailman/options/python-dev/steve.dower%40microsoft.com</a><br>

</div>

</span></font>

</body>

</html>