r219818 - Drop unneccessary default case from switch introduced in r219809

David Blaikie dblaikie at gmail.com
Wed Oct 15 11:05:08 PDT 2014


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/63f94485/attachment.html>


More information about the cfe-commits mailing list