[PATCH 3 of 3 V2] obsolete: drop the explicit seek to EOF after append mode open()
Matt Harbison
mharbison72 at gmail.com
Thu Feb 5 21:19:16 CST 2015
# HG changeset patch
# User Matt Harbison <matt_harbison at yahoo.com>
# Date 1422726875 18000
# Sat Jan 31 12:54:35 2015 -0500
# Node ID 949d3f3f9a3f189cc80cc8d5e4e1e875c60bd59e
# Parent 46fe95366461159cb59b4380350255f2933d75ed
obsolete: drop the explicit seek to EOF after append mode open()
posixfile now handles this.
diff --git a/mercurial/obsolete.py b/mercurial/obsolete.py
--- a/mercurial/obsolete.py
+++ b/mercurial/obsolete.py
@@ -77,8 +77,6 @@
_calcsize = struct.calcsize
propertycache = util.propertycache
-_SEEK_END = 2 # os.SEEK_END was introduced in Python 2.5
-
# the obsolete feature is not mature enough to be enabled by default.
# you have to rely on third party extension extension to enable this.
_enabled = False
@@ -592,12 +590,6 @@
if new:
f = self.sopener('obsstore', 'ab')
try:
- # Whether the file's current position is at the begin or at
- # the end after opening a file for appending is implementation
- # defined. So we must seek to the end before calling tell(),
- # or we may get a zero offset for non-zero sized files on
- # some platforms (issue3543).
- f.seek(0, _SEEK_END)
offset = f.tell()
transaction.add('obsstore', offset)
# offset == 0: new file - add the version header
More information about the Mercurial-devel
mailing list