<div class="gmail_quote">On Tue, Nov 25, 2008 at 1:21 PM, Ian Dees <span dir="ltr"><<a href="mailto:ian.dees@gmail.com">ian.dees@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
<div class="Ih2E3d">On Tue, Nov 25, 2008 at 1:50 PM, Patrick Kilian <span dir="ltr"><<a href="mailto:osm@petschge.de" target="_blank">osm@petschge.de</a>></span> wrote:<br></div><div class="gmail_quote"><div class="Ih2E3d">
<blockquote class="gmail_quote" style="border-left:1px solid rgb(204, 204, 204);margin:0pt 0pt 0pt 0.8ex;padding-left:1ex">
Hi all,<br>
<br>
I just implemented a new maplint test to check for tiger:reviewed=no<br>
tags. Currently it would generate notices (shown in yellow) because lots<br>
of objects are going to be highlighted by this test and I don't want to<br>
obscure other warnings. The new test can be seen at<br>
<a href="http://www.petschge.de/osm/maplint/maplint-tiger.patch" target="_blank">http://www.petschge.de/osm/maplint/maplint-tiger.patch</a> and is not in the<br>
svn yet. Are there any good reasons not to commit that test?</blockquote></div><div><br>There will be a *lot* of ways with reviewed=no. Almost all, in fact. I would rather see maplint show me the ways that are reviewed=yes.</div>
</div></blockquote><div><br></div><div>Personally, I think I would like to see notices for reviewed=no.  I suspect that once this gets added to maplint, people will pretty rapidly start to review and clear out this tag, especially in areas that are getting regular attention.</div>
<div><br></div><div>-Scott</div></div><br>-- <br>Scott Atwood<br><br>Cycle tracks will abound in Utopia.  ~H.G. Wells<br><br><br>