<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=Windows-1252">
</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><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>ext Ryosuke Niwa &lt;<a href="mailto:rniwa@webkit.org">rniwa@webkit.org</a>&gt;<br>
</div>
<div>
<div>
<div class="gmail_quote">
<div>&gt; r&#43; and r- flags are supposed to be set only by reviewers. If you wanted to withdraw your patch from the review queue, then you should be clearing &nbsp;r? flag, instead of setting r-. If you’re uploading a WIP patch, then it should not bear either r?, r-,
 or r&#43;</div>
</div>
</div>
</div>
</span><span id="OLK_SRC_BODY_SECTION">
<div>
<div>
<div class="gmail_quote">
<div>&gt; flags. You can accomplish this by either not setting the flag when you upload a patch on Bugzilla, clearing flag on the Bugzilla, or using --no-review option on webkit-patch.</div>
</div>
</div>
</div>
</span>
<div><br>
</div>
<div>Regarding WIP patches, what I've seen a few times is us reviewers adding an r- flag to a WIP patch with no r?, when we think it's horribly wrong…&nbsp;</div>
<div>I think the flip side of the guideline for non-reviewers to avoid r- is to have reviewers use r- only when the patch is up for review. This will encourage people to use no flags instead of putting r- for WIP patches.</div>
<div>No'am</div>
<span id="OLK_SRC_BODY_SECTION">
<div>
<div>
<div class="gmail_quote">
<div><br>
</div>
</div>
</div>
</div>
</span>
</body>
</html>