<div>
Sorry to resurrect this thread but is the consensus now that this line does not need to be removed? I apologize if I submitted faulty information before, I was simply going off what I witnessed another reviewer marking as required in their reviews (which is when I started doing it myself). I am getting the impression that this line does not need to be removed like I originally thought, am I correct in that assumption?
</div><div><br></div><div>Thanks. </div>
<div><br><!--?xml version="1.0" encoding="UTF-8"?-->
<span class="Apple-style-span" style="font-family: Arial; font-size: medium; "><span style="border-collapse: separate; color: rgb(0, 0, 0); font-family: Times; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: 2; text-align: -webkit-auto; text-indent: 0px; text-transform: none; white-space: normal; widows: 2; word-spacing: 0px; -webkit-border-horizontal-spacing: 0px; -webkit-border-vertical-spacing: 0px; -webkit-text-decorations-in-effect: none; -webkit-text-size-adjust: auto; -webkit-text-stroke-width: 0px; font-size: medium; "><span style="font-family: Helvetica; font-size: 13px; "><span>Regards,</span><span><br><br></span></span></span></span><div style="font-size: medium; font-family: Arial; "><span>Tyler Cunningham | Founder, COO - <a href="http://CyberChimps.com/" target="_blank" style="color: rgb(0, 58, 138); ">CyberChimps LLC</a></span></div><div style="font-size: medium; font-family: Arial; "><span><br></span></div><div style="font-size: medium; font-family: Arial; "><span><a href="http://twitter.com/tylerbcunning">@tylerbcunning</a></span></div><div style="font-size: medium; font-family: Arial; "><span><a href="http://gplus.to/tylercunningham">http://gplus.to/tylercunningham</a></span></div><div style="font-size: medium; font-family: Arial; "><span><a href="http://linkedin.com/in/tylerbcunningham" target="_blank" style="color: rgb(0, 58, 138); ">http://linkedin.com/in/tylerbcunningham</a> </span></div><div style="font-size: medium; font-family: Arial; "><span><a href="mailto:tyler@cyberchimps.com" target="_blank" style="color: rgb(0, 58, 138); ">tyler@cyberchimps.com</a></span></div><br></div>
<p style="color: #A0A0A8;">On Sunday, September 25, 2011 at 2:40 AM, Mike Little wrote:</p>
<blockquote type="cite" style="border-left-style:solid;border-width:1px;margin-left:0px;padding-left:10px;">
<span><div><div>Oops, accidently pressed send...<div><br></div><div><span style="color: rgb(34, 34, 34); font-family: arial, sans-serif; font-size: 13px; background-color: rgb(255, 255, 255); "><div>This line has also been in every version of WordPress since 0.7 </div>
<div>Historically, the file wp-comments.php used to be in the root of the WordPress directory (i.e. in the same place as wp-config.php.</div><div><br></div><div>And was probably instigated as it was one of the few files that would produce output if requested directly. The line protects against that.</div>
<div><br></div><div>Even when the theme files moved into their own directory in 1.5 the default (Kubrick), copeid the same code. Intrerestingly, classic dropped it.</div></span><font color="#222222" face="arial, sans-serif"><br>
<span style="font-size: 14px; background-color: rgb(255, 255, 255); ">In reality we should be cautious of any files that can produce output if requested directly, especially if they might produce errors. As this can be a source of information leak, and thus indirectly a security risk.</span><br>
</font><span style="color: rgb(34, 34, 34); font-family: arial, sans-serif; font-size: 13px; background-color: rgb(255, 255, 255); "><div><br></div><div>Mike</div></span>-- <br>Mike Little<div><a href="http://zed1.com/" target="_blank">http://zed1.com/</a></div>
<div><br></div><br>
</div>
</div><div>_______________________________________________<br>theme-reviewers mailing list<br><a href="mailto:theme-reviewers@lists.wordpress.org">theme-reviewers@lists.wordpress.org</a><br><a href="http://lists.wordpress.org/mailman/listinfo/theme-reviewers">http://lists.wordpress.org/mailman/listinfo/theme-reviewers</a><br></div></div></span>
</blockquote>
<div>
<br>
</div>