[PATCH] Minor comment and doc fixes

Jordan Rose jordan_rose at apple.com
Thu Sep 12 17:47:57 PDT 2013


Thanks, Jared...committed in r190651-2.

Jordan


On Sep 10, 2013, at 16:37 , Jared Grubb <jared.grubb at gmail.com> wrote:

> This is my first patch for clang, so I thought I'd make it a simple one. :)
> 
> As I've been learning the clang source, I found a couple small typos in the docs/comments. This patch is just a few lines of change:
> 
> * IPA.txt : a copy-paste error, referencing the wrong config flag (the real 'c++template-inlining' flag is the section right before this one)
> * Expr.h : 'getStrData' is a member function that no longer exists (removed in 2010 by a quick history search?); replaced with 'getBytes' to correspond with the 'getByteLength' referenced a few words later.
> * Idempotent : the entries for "x || x" and "x && x" are incorrect. I did not find any corresponding mistake in actual source, so this looks to just be a comment bug only.
> 
> I increased the context lines in the diff because I think it helps these patches.
> 
> Jared
> 
-------------- next part --------------
A non-text attachment was scrubbed...
Name: diff-minor-doc-fix.patch
Type: application/octet-stream
Size: 4581 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20130912/7f39256d/attachment.obj>
-------------- next part --------------
> _______________________________________________
> cfe-commits mailing list
> cfe-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits



More information about the cfe-commits mailing list