r219818 - Drop unneccessary default case from switch introduced in r219809

Kaelyn Takata rikka at google.com
Wed Oct 15 11:14:00 PDT 2014


Forgot about that! Added in r219820

On Wed, Oct 15, 2014 at 11:05 AM, David Blaikie <dblaikie at gmail.com> wrote:

> This'll probably end up with a GCC warning about missing return after the
> switch & will probably require an llvm_unreachable be placed after the
> switch to silence GCC.
>
> On Wed, Oct 15, 2014 at 10:46 AM, Kaelyn Takata <rikka at google.com> wrote:
>
>> Author: rikka
>> Date: Wed Oct 15 12:46:18 2014
>> New Revision: 219818
>>
>> URL: http://llvm.org/viewvc/llvm-project?rev=219818&view=rev
>> Log:
>> Drop unneccessary default case from switch introduced in r219809
>>
>> This silences:
>> ../tools/clang/tools/libclang/CIndex.cpp:6451:3: warning: default label
>> in switch which covers all enumeration values [-Wcovered-switch-default]
>>
>> Modified:
>>     cfe/trunk/tools/libclang/CIndex.cpp
>>
>> Modified: cfe/trunk/tools/libclang/CIndex.cpp
>> URL:
>> http://llvm.org/viewvc/llvm-project/cfe/trunk/tools/libclang/CIndex.cpp?rev=219818&r1=219817&r2=219818&view=diff
>>
>> ==============================================================================
>> --- cfe/trunk/tools/libclang/CIndex.cpp (original)
>> +++ cfe/trunk/tools/libclang/CIndex.cpp Wed Oct 15 12:46:18 2014
>> @@ -6448,8 +6448,6 @@ enum CX_StorageClass clang_Cursor_getSto
>>      return CX_SC_Auto;
>>    case SC_Register:
>>      return CX_SC_Register;
>> -  default:
>> -    return CX_SC_Invalid;
>>    }
>>  }
>>
>>
>>
>> _______________________________________________
>> cfe-commits mailing list
>> cfe-commits at cs.uiuc.edu
>> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
>>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20141015/57c98ff7/attachment.html>


More information about the cfe-commits mailing list