r181140 - ArrayRef<T>() -> None cleanup
David Blaikie
dblaikie at gmail.com
Mon May 6 09:54:26 PDT 2013
On Mon, May 6, 2013 at 9:47 AM, Jordan Rose <jordan_rose at apple.com> wrote:
> If it matters, there's another one in the ternary expression on the next line. ;-)
Subtly enough that one's necessary because FixIt isn't an ArrayRef,
but is convertible to ArrayRef. Without at least one of the operands
being an explicit/actual ArrayRef, it won't compile. It could be
rewritten as "? makeArrayRef(FixIt) : None" though, for example.
>
>
> On May 4, 2013, at 18:03 , Dmitri Gribenko <gribozavr at gmail.com> wrote:
>
>> Author: gribozavr
>> Date: Sat May 4 20:03:47 2013
>> New Revision: 181140
>>
>> URL: http://llvm.org/viewvc/llvm-project?rev=181140&view=rev
>> Log:
>> ArrayRef<T>() -> None cleanup
>>
>> Modified:
>> cfe/trunk/utils/TableGen/ClangDiagnosticsEmitter.cpp
>>
>> Modified: cfe/trunk/utils/TableGen/ClangDiagnosticsEmitter.cpp
>> URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/utils/TableGen/ClangDiagnosticsEmitter.cpp?rev=181140&r1=181139&r2=181140&view=diff
>> ==============================================================================
>> --- cfe/trunk/utils/TableGen/ClangDiagnosticsEmitter.cpp (original)
>> +++ cfe/trunk/utils/TableGen/ClangDiagnosticsEmitter.cpp Sat May 4 20:03:47 2013
>> @@ -245,7 +245,7 @@ static void groupDiagnostics(const std::
>> SourceMgr::DK_Error,
>> Twine("group '") + Name +
>> "' is referred to anonymously",
>> - ArrayRef<SMRange>(),
>> + None,
>> InGroupRange.isValid() ? FixIt
>> : ArrayRef<SMFixIt>());
>> SrcMgr.PrintMessage((*I)->ExplicitDef->getLoc().front(),
>>
>>
>> _______________________________________________
>> cfe-commits mailing list
>> cfe-commits at cs.uiuc.edu
>> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
>
> _______________________________________________
> 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