<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=us-ascii">
</head>
<body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; color: rgb(0, 0, 0); font-size: 14px; font-family: Calibri, sans-serif; ">
<div>Oh, many thanks to your guys! </div>
<div><br>
</div>
<div>Huayang</div>
<div><br>
</div>
<span id="OLK_SRC_BODY_SECTION">
<div style="font-family:Calibri; font-size:11pt; text-align:left; color:black; BORDER-BOTTOM: medium none; BORDER-LEFT: medium none; PADDING-BOTTOM: 0in; PADDING-LEFT: 0in; PADDING-RIGHT: 0in; BORDER-TOP: #b5c4df 1pt solid; BORDER-RIGHT: medium none; PADDING-TOP: 3pt">
<span style="font-weight:bold">From: </span>Augie Fackler <<a href="mailto:raf@durin42.com">raf@durin42.com</a>><br>
<span style="font-weight:bold">Date: </span>Wed, 17 Oct 2012 19:51:26 -0500<br>
<span style="font-weight:bold">To: </span>Huayang Guo <<a href="mailto:huayang@fb.com">huayang@fb.com</a>><br>
<span style="font-weight:bold">Cc: </span>Mads Kiilerich <<a href="mailto:mads@kiilerich.com">mads@kiilerich.com</a>>, Mercurial-devel <<a href="mailto:mercurial-devel@selenic.com">mercurial-devel@selenic.com</a>>, Bryan O'Sullivan <<a href="mailto:bryano@fb.com">bryano@fb.com</a>>,
 Matt Mackall <<a href="mailto:mpm@selenic.com">mpm@selenic.com</a>><br>
<span style="font-weight:bold">Subject: </span>Re: [PATCH] convert: normalize paths in filemaps (issue3612)<br>
</div>
<div><br>
</div>
<p dir="ltr">I'd interpret mpm's message as indicating he did the cleanup. </p>
<div class="gmail_quote">On Oct 17, 2012 7:05 PM, "Huayang Guo" <<a href="mailto:huayang@fb.com">huayang@fb.com</a>> wrote:<br type="attribution">
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Hi, Matt,<br>
<br>
Thanks for your review. Just wondering if I should submit another diff to<br>
revise the issues given by contrib/check-node.py?<br>
<br>
Thanks,<br>
<br>
Huayang<br>
<br>
On 10/17/12 4:10 PM, "Matt Mackall" <<a href="mailto:mpm@selenic.com">mpm@selenic.com</a>> wrote:<br>
<br>
>On Wed, 2012-10-17 at 14:01 -0700, Huayang Guo wrote:<br>
>> # HG changeset patch<br>
>> # User Huayang <<a href="mailto:huayang@fb.com">huayang@fb.com</a>><br>
>> # Date 1349479654 25200<br>
>> # Node ID be43ef5cd839673f76dd0b65a9f053a6e3ac76af<br>
>> # Parent  fa714f3ed2989aff64c267c9935251d9fc4f31ee<br>
>> convert: normalize paths in filemaps (issue3612)<br>
><br>
>Queued for default with some fixes. contrib/check-code.py has asked me<br>
>to tell you:<br>
><br>
>hgext/convert/filemap.py:21:<br>
> >     ''' We use posixpath.normpath to support cross-platform path<br>
>format.<br>
> trailing whitespace<br>
>hgext/convert/filemap.py:23:<br>
> >     if path == None:<br>
> comparison with singleton, use 'is' or 'is not' instead<br>
><br>
>--<br>
>Mathematics is the supreme nostalgia of our time.<br>
><br>
><br>
<br>
</blockquote>
</div>
</span>
</body>
</html>