
<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;"><br><div><div>On Jul 31, 2014, at 12:21 PM, Aaron Ballman <<a href="mailto:aaron@aaronballman.com">aaron@aaronballman.com</a>> wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite">The attach patch addresses a FIXME asking for improved source range<br>information when a nonnull parameter or return type is not a pointer<br>type. A few questions come to mind:<br><br>1) Is my addition to ObjCMethodDecl the correct way to get return type<br>source range information?<br></blockquote><div><br></div>LGTM. We use the same in other places.For example, in <span style="font-family: Menlo; font-size: 11px; color: rgb(79, 129, 135);">Sema</span><span style="font-family: Menlo; font-size: 11px;">::CheckObjCMethodOverride</span></div><div><font face="Menlo"><span style="font-size: 11px;">- Fariborz</span></font></div><div><font face="Menlo"><span style="font-size: 11px;"><br></span></font><blockquote type="cite">2) Does this patch require tests? If so, do we have any source range<br>tests I should model after?<br><br>Thanks!<br><br>~Aaron<br><span><nonnull.patch></span></blockquote></div><br></body></html>