msg57147 - (view) |
Author: Amaury Forgeot d'Arc (amaury.forgeotdarc) * |
Date: 2007-11-06 00:21 |
When reading a Windows text file one byte at a time, \r\n get split into
two function calls, and the user receives two \n.
The following test fails (put it somewhere in test_io.py, inside
TextIOWrapperTest for example)
def testReadOneByOne(self):
txt = io.TextIOWrapper(io.BytesIO(b"AA\r\nBB"))
reads = ""
while True:
c = txt.read(1)
if not c:
break
reads += c
self.assertEquals(reads, "AA\nBB")
# AssertionError: 'AA\n\nBB' != 'AA\nBB'
Note that replacing read(1) by read(2) gives the correct result.
This problem is why test_netrc fails on Windows. It may also be the root
cause for issue 1142 (when \r\n position is just a multiple of the
_CHUNK_SIZE).
It also possible that the correction to this problem will have good
effects on test_mailbox, which uses tell() and seek() intensively.
|
msg57152 - (view) |
Author: Guido van Rossum (gvanrossum) * |
Date: 2007-11-06 00:46 |
Wow, thanks!
This is not just a bug on Windows -- it is a bug in the TextIOWrapper
code that is just more likely on Windows. It is easily reproduced on
Linux too:
>>> f = open("@", "wb")>>> f.write(b"a\r\n")
6
>>> f.close()
>>> f = open("@", "r")
>>> f.read(1)
'a'
>>> f.read(1)
'\n'
>>>
|
msg57155 - (view) |
Author: Raghuram Devarakonda (draghuram) |
Date: 2007-11-06 04:36 |
I think the solution is to do the translation on a bigger chunk than on
the string being returned in each read call. For example, the following
patch correctly returns "a" and "\n" (instead of "a" and two "\n"s).
Index: Lib/io.py
===================================================================
--- Lib/io.py (revision 58874)
+++ Lib/io.py (working copy)
@@ -1253,8 +1253,9 @@
res += pending
if not readahead:
break
+ res = self._replacenl(res)
self._pending = res[n:]
- return self._replacenl(res[:n])
+ return res[:n]
def __next__(self):
self._telling = False
-----------------
Of course, we need to take care of the case when the last character in
"res" is "\r" to be followed by "\n" in the next read. I just wanted to
see if I am on the right track and if so, I can spend more time on this.
|
msg57156 - (view) |
Author: Amaury Forgeot d'Arc (amaury.forgeotdarc) * |
Date: 2007-11-06 08:11 |
Some thoughts:
- is it ok to call replacenl multiple times on the same string? \r\r\n
and other combinations like this.
- I wonder if it is possible to correctly handle \r\n at the CHUNK_SIZE
limit without an incremental decoder. it seems that we need at least a
flag saying "\r was previously read, ignore the next \n"
|
msg57163 - (view) |
Author: Raghuram Devarakonda (draghuram) |
Date: 2007-11-06 16:12 |
I am attaching the patch io.diff that does the following:
- If a chunk ends in "\r", read the next chunk to check if it starts
with "\n". This is obviously a very simplified solution that can be
optimized.
- invoke _replacenl on the complete string read instead of what is being
returned in each read call.
I also incorporated the test case by Amaury and added one more.
With this patch in place, the following tests failed (on SuSE 10.1):
test_doctest test_mailbox test_nis test_old_mailbox
test_pickletools test_pty test_sys
The failures (other than known test_mailbox and test_old_mailbox) didn't
look like they are caused by this fix.
|
msg57164 - (view) |
Author: Amaury Forgeot d'Arc (amaury.forgeotdarc) * |
Date: 2007-11-06 16:51 |
This patch goes in the right direction, but has several problems IMO:
- it reads a complete chunk for just one more byte
- the re-read should be disabled when lineends are not translated
these two are minor annoyance and can be easily corrected, but:
- there is no limit to the re-read; it can exhaust the memory if the
source is a big file with many \r (like a Mac text file)
- How does this behave if the underlying stream has not yet available
data (a socket, a terminal, or an opened file): will the re-read block
at the 129th byte until more data is available? If so, it is annoying
for a simple call to read(1).
I will try to write these test cases if you want.
|
msg57166 - (view) |
Author: Guido van Rossum (gvanrossum) * |
Date: 2007-11-06 17:43 |
IMO you shouldn't read another chunk when the last character you've seen
is \r; instead, you should set a flag so that on the next read, you'll
ignore an initial \n. The flag should be made of the tell/seek state as
well.
(The problem with reading another character is that in interactive input
mode, this might force the user to type an entire second line.)
|
msg57169 - (view) |
Author: Raghuram Devarakonda (draghuram) |
Date: 2007-11-06 17:52 |
On 11/6/07, Amaury Forgeot d'Arc <report@bugs.python.org> wrote:
> - it reads a complete chunk for just one more byte
> - the re-read should be disabled when lineends are not translated
> these two are minor annoyance and can be easily corrected, but:
>
> - there is no limit to the re-read; it can exhaust the memory if the
> source is a big file with many \r (like a Mac text file)
Yes. reading another chunk is not an optimal solution but after seeing
Guido's comment, I now agree that I shouldn't try to read at all. I
will work on modifications.
> I will try to write these test cases if you want.
That will definitely be useful.
|
msg57171 - (view) |
Author: Christian Heimes (christian.heimes) * |
Date: 2007-11-06 18:21 |
Guido van Rossum wrote:
> IMO you shouldn't read another chunk when the last character you've seen
> is \r; instead, you should set a flag so that on the next read, you'll
> ignore an initial \n. The flag should be made of the tell/seek state as
> well.
In my opinion the check for \r should only happen when os.linesep or
newline starts with \r. On Unix with standard newline the \r should be
treated as every other char.
Christian
|
msg57176 - (view) |
Author: Guido van Rossum (gvanrossum) * |
Date: 2007-11-06 18:59 |
> In my opinion the check for \r should only happen when os.linesep or
> newline starts with \r. On Unix with standard newline the \r should be
> treated as every other char.
No: it is not dependent on os.linesep but on the newline parameter
passed to open().
It's perfectly valid to want to read a file with CRLF endings on Unix
(Amaury's patches end up in my /tmp directory like that.)
|
msg57204 - (view) |
Author: Raghuram Devarakonda (draghuram) |
Date: 2007-11-07 17:08 |
I am attaching another patch (io2.diff). Please review. I am not sure
whether _adjust_chunk() should also adjust "readahead".
BTW, PEP 3116 says:
"If universal newlines without translation are requested on input (i.e.
newline=''), if a system read operation returns a buffer ending in '\r',
another system read operation is done to determine whether it is
followed by '\n' or not. In universal newlines mode with translation,
the second system read operation may be postponed until the next read
request, and if the following system read operation returns a buffer
starting with '\n', that character is simply discarded."
I suppose this issue is mainly talking about the latter (newline is
None). I don't understand what is meant by "enabling universal new line
mode without translation". Isn't the purpose of enabling universal new
line mode is to translate line endings? I may be missing something
basic, of course.
|
msg57205 - (view) |
Author: Christian Heimes (christian.heimes) * |
Date: 2007-11-07 17:20 |
The new patch fixes test_netrc for me but test_csv and test_mailbox are
still broken.
|
msg57207 - (view) |
Author: Raghuram Devarakonda (draghuram) |
Date: 2007-11-07 17:28 |
> The new patch fixes test_netrc for me but test_csv and test_mailbox are
> still broken.
Unfortunately, I am not able to build python on windows so I can not
test there. Can you post the exact errors? You can send me private
email as well if you think the test output would clutter this issue.
|
msg57208 - (view) |
Author: Guido van Rossum (gvanrossum) * |
Date: 2007-11-07 17:41 |
This looks promising, but my head hurts when I try to understand the
code that's already there and think about whether your patch will always
do the right thing... I'll look more later.
Regarding "universal newlines without translation:" that means that \r\n
and \r are recognized as line endings (as is \n) and that readline()
will return whatever line end it sees. Compare this to setting
newline="\n"; then \r is not treated as a line ending at all (and if the
input is a\rb\n, the next readline call would return that entire string).
|
msg57209 - (view) |
Author: Amaury Forgeot d'Arc (amaury.forgeotdarc) * |
Date: 2007-11-07 17:47 |
> The new patch fixes test_netrc for me but test_csv and test_mailbox are
> still broken.
For test_mailbox at least, I think I have a clue: the _pending member
now contains translated newlines.
But in tell(), we use its length and compare it with the length of a
previous "pending" value stored in self._snapshot...
Can we try to somehow move the replacenl() call inside the _get_chunk
function?
Not sure it will work. This gets more and more obscure...
|
msg57210 - (view) |
Author: Guido van Rossum (gvanrossum) * |
Date: 2007-11-07 17:49 |
Somebody needs to reverse-engineer the invariants applying to the
various instance variables and add comments explaining them, and
showing how they are maintained by each call.
|
msg57214 - (view) |
Author: Christian Heimes (christian.heimes) * |
Date: 2007-11-07 18:34 |
By the way what happened to the SoC project related to Python's new IO
subsystem? IIRC somebody was working on a C optimization of the io lib.
|
msg57217 - (view) |
Author: Guido van Rossum (gvanrossum) * |
Date: 2007-11-07 19:03 |
On 11/7/07, Christian Heimes <report@bugs.python.org> wrote:
>
> Christian Heimes added the comment:
>
> By the way what happened to the SoC project related to Python's new IO
> subsystem? IIRC somebody was working on a C optimization of the io lib.
>
> __________________________________
> Tracker <report@bugs.python.org>
> <http://bugs.python.org/issue1395>
> __________________________________
I think it was Alexandre Vassalotti. Is that right, Alexandre? Or am I
mixing you up? (If you ca, please respond to the bug.)
|
msg57218 - (view) |
Author: Raghuram Devarakonda (draghuram) |
Date: 2007-11-07 19:06 |
Hi Amaury and Christian,
io3.diff does replacenl() in adjust_chunk() (trying Amaury's
suggestion). Can you see if it fixes test_mailbox failures?
|
msg57219 - (view) |
Author: Christian Heimes (christian.heimes) * |
Date: 2007-11-07 19:10 |
Good work!
The tests for mailbox, netrc and csv are passing with your test. I'm
going to run the entire suite now.
|
msg57220 - (view) |
Author: Christian Heimes (christian.heimes) * |
Date: 2007-11-07 19:14 |
I take it back. I accidentally run the unit tests on the trunk instead
of the py3k branch. mailbox and csv are still breaking with your test,
netrc is doing fine.
|
msg57228 - (view) |
Author: Alexandre Vassalotti (alexandre.vassalotti) * |
Date: 2007-11-07 23:55 |
On 11/7/07, Guido van Rossum <guido@python.org> wrote:
> On 11/7/07, Christian Heimes <report@bugs.python.org> wrote:
> >
> > Christian Heimes added the comment:
> >
> > By the way what happened to the SoC project related to Python's new IO
> > subsystem? IIRC somebody was working on a C optimization of the io lib.
> >
>
> I think it was Alexandre Vassalotti. Is that right, Alexandre? Or am I
> mixing you up? (If you ca, please respond to the bug.)
I think so. My GSoC project was to merge the interface of
cPickle/pickle and cStringIO/StringIO. I am still working on it,
albeit slowly (my school homework is killing all my free time, right
now). My work on StringIO and BytesIO is basically done; I just need
to add newline translation and encoding support before it can be
merged into the trunk. The cPickle/pickle merge is mostly done (i.e.,
all the current tests passes); I am right now in the bug-hunting
phase.
|
msg57229 - (view) |
Author: Guido van Rossum (gvanrossum) * |
Date: 2007-11-08 00:11 |
Cool. How hard do you think it would be to extend your work on
StringIO into a translation of TextIOWrapper into C?
On Nov 7, 2007 3:55 PM, Alexandre Vassalotti <alexandre@peadrop.com> wrote:
> On 11/7/07, Guido van Rossum <guido@python.org> wrote:
> > On 11/7/07, Christian Heimes <report@bugs.python.org> wrote:
> > >
> > > Christian Heimes added the comment:
> > >
> > > By the way what happened to the SoC project related to Python's new IO
> > > subsystem? IIRC somebody was working on a C optimization of the io lib.
> > >
> >
> > I think it was Alexandre Vassalotti. Is that right, Alexandre? Or am I
> > mixing you up? (If you ca, please respond to the bug.)
>
> I think so. My GSoC project was to merge the interface of
> cPickle/pickle and cStringIO/StringIO. I am still working on it,
> albeit slowly (my school homework is killing all my free time, right
> now). My work on StringIO and BytesIO is basically done; I just need
> to add newline translation and encoding support before it can be
> merged into the trunk. The cPickle/pickle merge is mostly done (i.e.,
> all the current tests passes); I am right now in the bug-hunting
> phase.
>
|
msg57230 - (view) |
Author: Amaury Forgeot d'Arc (amaury.forgeotdarc) * |
Date: 2007-11-08 00:30 |
> io3.diff does replacenl() in adjust_chunk() (trying Amaury's
> suggestion). Can you see if it fixes test_mailbox failures?
Unfortunately, it does not. And some tests now fail in test_io
(correcting testTelling seems a good start point, since we just changed
the tell() behaviour)
|
msg57231 - (view) |
Author: Alexandre Vassalotti (alexandre.vassalotti) * |
Date: 2007-11-08 01:51 |
On 11/7/07, Guido van Rossum wrote:
> Cool. How hard do you think it would be to extend your work on
> StringIO into a translation of TextIOWrapper into C?
Well, StringIO and TextIOWrapper are quite different. The only part
that I could reuse, from StringIO, would be the newline translation
facilities. I think that a perfect translation to C of the current
Python implementation of TextIOWrapper will be burdensome (due to
things like function annotations) but not too hard to do.
Nevertheless, that would be neat project for me. I could start to work
it by mid-December, along with the other performance enhancements I
have in mind.
|
msg57234 - (view) |
Author: Amaury Forgeot d'Arc (amaury.forgeotdarc) * |
Date: 2007-11-08 12:59 |
OK, I have taken another approach which seems to work (see io4.diff):
It uses an IncrementalNewlineDecoder, which wraps the initial (e.g.
utf-8) decoder.
All the tests in test_io pass on Windows, including those added by
io.diff and io2.diff. This was not the case with the other proposed patches.
While not completely finished, this approach seems much saner to me: it
is simple, does not introduce obscure variables or functions, and is
compatible with the (complex) code in tell() which reconstruct the file
position.
Next steps are:
- move _seennl management inside this decoder, and remove _replacenl
- make the decoder directly inherit from codecs.IncrementalDecoder; code
may be easier to understand.
- fix test_mailbox.
About mailbox.py: it seems that the code cannot work: it uses statements
like
self._file.read(stop - self._file.tell())
where 'stop' was previously initialized with some other call to
self._file.tell(). But read() counts characters, whereas tell() returns
byte counts. The question is now: how does it work with python2.5? I'll
try to add debug prints to see where the differences are.
|
msg57235 - (view) |
Author: Christian Heimes (christian.heimes) * |
Date: 2007-11-08 13:17 |
The patch doesn't apply
$ patch -p0 < io4.diff
(Stripping trailing CRs from patch.)
patching file Lib/io.py
patch: **** malformed patch at line 41: @@ -1133,7 +1160,10 @@
|
msg57236 - (view) |
Author: Amaury Forgeot d'Arc (amaury.forgeotdarc) * |
Date: 2007-11-08 13:23 |
Sorry, I think I corrupted the file by hand. Here is another version
|
msg57238 - (view) |
Author: Amaury Forgeot d'Arc (amaury.forgeotdarc) * |
Date: 2007-11-08 13:30 |
> About mailbox.py: it seems that the code cannot work
Of course: the file mode was recently changed from rb+ to r+ (revision
57809). This means that every occurrence of os.linesep has to disappear.
Oh my.
|
msg57252 - (view) |
Author: Christian Heimes (christian.heimes) * |
Date: 2007-11-08 14:40 |
The new io4.diff breaks test_io and test_univnewlines on Linux
|
msg57264 - (view) |
Author: Amaury Forgeot d'Arc (amaury.forgeotdarc) * |
Date: 2007-11-08 17:37 |
> The new io4.diff breaks test_io and test_univnewlines on Linux
oops, I missed this one.
Here is a new version: io5.diff, which should handle the "seen newlines"
better.
Two more bug fixes found by test_univnewlines:
- write() should return the number of characters written, but before
they are newline-translated.
- a problem in tell(), my decoder can return two characters even when
you feed only one (example: decode(b'\r') holds the \r and returns
nothing. then a decode(b'x') returns both chars "\rx")
|
msg57272 - (view) |
Author: Raghuram Devarakonda (draghuram) |
Date: 2007-11-08 18:54 |
Considering that test_csv is failing on windows even without any changes
related to this issue, I looked at it and came up with this patch:
-----------------
Index: Lib/test/test_csv.py
===================================================================
--- Lib/test/test_csv.py (revision 58914)
+++ Lib/test/test_csv.py (working copy)
@@ -375,7 +375,7 @@
class TestCsvBase(unittest.TestCase):
def readerAssertEqual(self, input, expected_result):
- with TemporaryFile("w+") as fileobj:
+ with TemporaryFile("w+", newline='') as fileobj:
fileobj.write(input)
fileobj.seek(0)
reader = csv.reader(fileobj, dialect = self.dialect)
-----------------
Does this look ok? The tests pass on windows and Linux.
|
msg57273 - (view) |
Author: Raghuram Devarakonda (draghuram) |
Date: 2007-11-08 19:00 |
On 11/8/07, Amaury Forgeot d'Arc <report@bugs.python.org> wrote:
> OK, I have taken another approach which seems to work (see io4.diff):
> It uses an IncrementalNewlineDecoder, which wraps the initial (e.g.
> utf-8) decoder.
I like this approach even though I haven't looked at the patch in detail.
|
msg57283 - (view) |
Author: Amaury Forgeot d'Arc (amaury.forgeotdarc) * |
Date: 2007-11-08 23:30 |
Updated patch (io6.diff):
- simplifications in readline
- seennl is now completely handled by the NewlineDecoder
|
msg57291 - (view) |
Author: Guido van Rossum (gvanrossum) * |
Date: 2007-11-09 00:42 |
> Considering that test_csv is failing on windows even without any changes
> related to this issue, I looked at it and came up with this patch:
>
> -----------------
> Index: Lib/test/test_csv.py
> ===================================================================
> --- Lib/test/test_csv.py (revision 58914)
> +++ Lib/test/test_csv.py (working copy)
> @@ -375,7 +375,7 @@
>
> class TestCsvBase(unittest.TestCase):
> def readerAssertEqual(self, input, expected_result):
> - with TemporaryFile("w+") as fileobj:
> + with TemporaryFile("w+", newline='') as fileobj:
> fileobj.write(input)
> fileobj.seek(0)
> reader = csv.reader(fileobj, dialect = self.dialect)
> -----------------
>
> Does this look ok? The tests pass on windows and Linux.
Yes, especially since writerAssertEqual() already uses that. :-)
I do think there is something iffy here -- the 2.x version of this
test opens the files in binary mode. I wonder what end users are
supposed to do.
|
msg57311 - (view) |
Author: Raghuram Devarakonda (draghuram) |
Date: 2007-11-09 15:50 |
On 11/8/07, Guido van Rossum <report@bugs.python.org> wrote:
> I do think there is something iffy here -- the 2.x version of this
> test opens the files in binary mode. I wonder what end users are
> supposed to do.
I think that requirement (need to open in binary mode) is no more
present in 3.0. As per revision 56777 (#1767398), csv module seems to
support unicode strings and as such I would expect that files *should
not* be opened in binary mode (This requires doc change both in csv
doc and "what's new in 3.0). The tests in test_csv are explicitly
writing "\r\n" which seems to be a hangover from 2.x. We seem to have
side-stepped the problem by passing "newline=''" at some places but
the real fix may be changing "\r\n"s to "\n" now that the temp files
are being opened in text mode.
|
msg57368 - (view) |
Author: Christian Heimes (christian.heimes) * |
Date: 2007-11-11 15:09 |
By the way I've found the daily builds you were asking for, Raghuram.
http://www.python.org/dev/daily-msi/ :)
|
msg57664 - (view) |
Author: Amaury Forgeot d'Arc (amaury.forgeotdarc) * |
Date: 2007-11-19 20:35 |
Committed the patch in r59060.
|
msg148182 - (view) |
Author: Sébastien Sablé (sable) |
Date: 2011-11-23 13:47 |
I am trying to get Python working when compiled with Visual Studio 2010 (cf issue 13210).
When running the tests with the python 2.7 branch compiled with VS2010, the "test_issue_1395_5" in test_io.py will cause Python to eat the whole memory within a few seconds and make the server completely unresponsive.
|
msg148183 - (view) |
Author: Amaury Forgeot d'Arc (amaury.forgeotdarc) * |
Date: 2011-11-23 13:51 |
You should open a new issue for this new problem.
|
msg148185 - (view) |
Author: Sébastien Sablé (sable) |
Date: 2011-11-23 14:03 |
OK, sorry. Done in issue 13461.
|
|
Date |
User |
Action |
Args |
2022-04-11 14:56:28 | admin | set | github: 45736 |
2011-11-23 14:03:38 | sable | set | messages:
+ msg148185 |
2011-11-23 13:51:11 | amaury.forgeotdarc | set | messages:
+ msg148183 |
2011-11-23 13:48:43 | vstinner | set | nosy:
+ vstinner
|
2011-11-23 13:47:42 | sable | set | nosy:
+ sable messages:
+ msg148182
|
2008-01-06 22:29:45 | admin | set | keywords:
- py3k versions:
Python 3.0 |
2007-11-19 20:35:55 | amaury.forgeotdarc | set | status: open -> closed resolution: fixed messages:
+ msg57664 |
2007-11-11 15:09:51 | christian.heimes | set | messages:
+ msg57368 |
2007-11-09 15:50:32 | draghuram | set | messages:
+ msg57311 |
2007-11-09 00:42:13 | gvanrossum | set | messages:
+ msg57291 |
2007-11-08 23:30:01 | amaury.forgeotdarc | set | files:
+ io6.diff messages:
+ msg57283 |
2007-11-08 19:00:25 | draghuram | set | messages:
+ msg57273 |
2007-11-08 18:54:25 | draghuram | set | messages:
+ msg57272 |
2007-11-08 17:37:27 | amaury.forgeotdarc | set | files:
+ io5.diff messages:
+ msg57264 |
2007-11-08 14:40:02 | christian.heimes | set | files:
+ linux_test.log.gz messages:
+ msg57252 |
2007-11-08 13:30:48 | amaury.forgeotdarc | set | messages:
+ msg57238 |
2007-11-08 13:23:36 | amaury.forgeotdarc | set | files:
+ io4.diff messages:
+ msg57236 |
2007-11-08 13:17:29 | christian.heimes | set | messages:
+ msg57235 |
2007-11-08 12:59:09 | amaury.forgeotdarc | set | files:
+ io4.diff messages:
+ msg57234 |
2007-11-08 01:51:59 | alexandre.vassalotti | set | messages:
+ msg57231 |
2007-11-08 00:30:48 | amaury.forgeotdarc | set | messages:
+ msg57230 |
2007-11-08 00:11:54 | gvanrossum | set | messages:
+ msg57229 |
2007-11-07 23:55:40 | alexandre.vassalotti | set | messages:
+ msg57228 |
2007-11-07 19:14:56 | christian.heimes | set | files:
+ py3k_windows.log.gz messages:
+ msg57220 |
2007-11-07 19:10:59 | christian.heimes | set | messages:
+ msg57219 |
2007-11-07 19:07:05 | christian.heimes | set | nosy:
+ alexandre.vassalotti |
2007-11-07 19:06:05 | draghuram | set | files:
+ io3.diff messages:
+ msg57218 |
2007-11-07 19:03:08 | gvanrossum | set | messages:
+ msg57217 |
2007-11-07 18:34:59 | christian.heimes | set | messages:
+ msg57214 |
2007-11-07 17:49:33 | gvanrossum | set | messages:
+ msg57210 |
2007-11-07 17:47:22 | amaury.forgeotdarc | set | messages:
+ msg57209 |
2007-11-07 17:41:31 | gvanrossum | set | messages:
+ msg57208 |
2007-11-07 17:28:33 | draghuram | set | messages:
+ msg57207 |
2007-11-07 17:20:30 | christian.heimes | set | keywords:
+ py3k messages:
+ msg57205 components:
+ Library (Lib), Windows |
2007-11-07 17:08:57 | draghuram | set | files:
+ io2.diff messages:
+ msg57204 |
2007-11-06 18:59:14 | gvanrossum | set | messages:
+ msg57176 |
2007-11-06 18:21:09 | christian.heimes | set | messages:
+ msg57171 |
2007-11-06 17:52:23 | draghuram | set | messages:
+ msg57169 |
2007-11-06 17:43:35 | gvanrossum | set | messages:
+ msg57166 |
2007-11-06 16:51:50 | amaury.forgeotdarc | set | messages:
+ msg57164 |
2007-11-06 16:12:18 | draghuram | set | files:
+ io.diff messages:
+ msg57163 |
2007-11-06 08:11:58 | amaury.forgeotdarc | set | messages:
+ msg57156 |
2007-11-06 04:36:32 | draghuram | set | nosy:
+ draghuram messages:
+ msg57155 |
2007-11-06 02:18:37 | christian.heimes | link | issue1345 superseder |
2007-11-06 00:46:20 | gvanrossum | set | priority: high messages:
+ msg57152 |
2007-11-06 00:21:36 | amaury.forgeotdarc | create | |