# HG changeset patch # User Pierre-Yves David # Date 1393534781 28800 # Node ID 31993cd23b11334e1f323b2e4488ad038d4b4137 # Parent 4c4cd64c754b4bad3876423ec11c2ccb0a74ba1d merge: change the merge state serialisation to use a record based logic The format of the file is unchanged. But we are preparing a new file with a new format that would be record based. So we change all the read/write logic to handle a list of record until a very low level. This will allow simple plugging of the new format in the current code. diff -r 4c4cd64c754b -r 31993cd23b11 mercurial/merge.py --- a/mercurial/merge.py Tue Feb 25 17:14:49 2014 -0800 +++ b/mercurial/merge.py Thu Feb 27 12:59:41 2014 -0800 @@ -26,27 +26,49 @@ self._dirty = False def _read(self): self._state = {} + records = self._readrecords() + for rtype, record in records: + if rtype == 'L': + self._local = bin(record) + elif rtype == "F": + bits = record.split("\0") + self._state[bits[0]] = bits[1:] + elif not rtype.islower(): + raise util.Abort(_('unsupported merge state record:' + % rtype)) + self._dirty = False + def _readrecords(self): + records = [] try: f = self._repo.opener(self.statepath) for i, l in enumerate(f): if i == 0: - self._local = bin(l[:-1]) + records.append(('L', l[:-1])) else: - bits = l[:-1].split("\0") - self._state[bits[0]] = bits[1:] + records.append(('F', l[:-1])) f.close() except IOError, err: if err.errno != errno.ENOENT: raise - self._dirty = False + return records def commit(self): if self._dirty: - f = self._repo.opener(self.statepath, "w") - f.write(hex(self._local) + "\n") + records = [] + records.append(("L", hex(self._local))) for d, v in self._state.iteritems(): - f.write("\0".join([d] + v) + "\n") - f.close() + records.append(("F", "\0".join([d] + v))) + self._writerecords(records) self._dirty = False + def _writerecords(self, records): + f = self._repo.opener(self.statepath, "w") + irecords = iter(records) + lrecords = irecords.next() + assert lrecords[0] == 'L' + f.write(hex(self._local) + "\n") + for rtype, data in irecords: + if rtype == "F": + f.write("%s\n" % data) + f.close() def add(self, fcl, fco, fca, fd): hash = util.sha1(fcl.path()).hexdigest() self._repo.opener.write("merge/" + hash, fcl.data())