[Lldb-commits] [lldb] r260914 - Silence some clang warnings

Saleem Abdulrasool via lldb-commits lldb-commits at lists.llvm.org
Mon Feb 15 13:50:29 PST 2016


Author: compnerd
Date: Mon Feb 15 15:50:28 2016
New Revision: 260914

URL: http://llvm.org/viewvc/llvm-project?rev=260914&view=rev
Log:
Silence some clang warnings

Silences -Wmissing-brace and -Wformat-pedantic warnings from clang on Linux.  NFC.

Modified:
    lldb/trunk/source/Plugins/LanguageRuntime/RenderScript/RenderScriptRuntime/RenderScriptRuntime.cpp
    lldb/trunk/tools/driver/Driver.cpp

Modified: lldb/trunk/source/Plugins/LanguageRuntime/RenderScript/RenderScriptRuntime/RenderScriptRuntime.cpp
URL: http://llvm.org/viewvc/llvm-project/lldb/trunk/source/Plugins/LanguageRuntime/RenderScript/RenderScriptRuntime/RenderScriptRuntime.cpp?rev=260914&r1=260913&r2=260914&view=diff
==============================================================================
--- lldb/trunk/source/Plugins/LanguageRuntime/RenderScript/RenderScriptRuntime/RenderScriptRuntime.cpp (original)
+++ lldb/trunk/source/Plugins/LanguageRuntime/RenderScript/RenderScriptRuntime/RenderScriptRuntime.cpp Mon Feb 15 15:50:28 2016
@@ -174,15 +174,15 @@ GetArgsX86_64(GetArgsCtx &ctx, ArgItem *
     // number of arguments passed in registers
     static const uint32_t c_args_in_reg = 6;
     // register passing order
-    static const std::array<const char *, c_args_in_reg> c_reg_names = {"rdi", "rsi", "rdx", "rcx", "r8", "r9"};
+    static const std::array<const char *, c_args_in_reg> c_reg_names{{"rdi", "rsi", "rdx", "rcx", "r8", "r9"}};
     // argument type to size mapping
-    static const std::array<size_t, 5> arg_size = {
+    static const std::array<size_t, 5> arg_size{{
         8, // ePointer,
         4, // eInt32,
         8, // eInt64,
         8, // eLong,
         4, // eBool,
-    };
+    }};
 
     // get the current stack pointer
     uint64_t sp = ctx.reg_ctx->GetSP();
@@ -1020,7 +1020,7 @@ RenderScriptRuntime::CaptureScriptInvoke
         eRsSc,
     };
 
-    std::array<ArgItem, 9> args = {
+    std::array<ArgItem, 9> args{{
         ArgItem{ArgItem::ePointer, 0}, // const Context       *rsc
         ArgItem{ArgItem::ePointer, 0}, // Script              *s
         ArgItem{ArgItem::eInt32, 0},   // uint32_t             slot
@@ -1030,7 +1030,7 @@ RenderScriptRuntime::CaptureScriptInvoke
         ArgItem{ArgItem::ePointer, 0}, // const void          *usr
         ArgItem{ArgItem::eInt32, 0},   // size_t               usrLen
         ArgItem{ArgItem::ePointer, 0}, // const RsScriptCall  *sc
-    };
+    }};
 
     bool success = GetArgs(context, &args[0], args.size());
     if (!success)
@@ -1125,13 +1125,13 @@ RenderScriptRuntime::CaptureSetGlobalVar
         eRsLength,
     };
 
-    std::array<ArgItem, 5> args = {
+    std::array<ArgItem, 5> args{{
         ArgItem{ArgItem::ePointer, 0}, // eRsContext
         ArgItem{ArgItem::ePointer, 0}, // eRsScript
         ArgItem{ArgItem::eInt32, 0},   // eRsId
         ArgItem{ArgItem::ePointer, 0}, // eRsData
         ArgItem{ArgItem::eInt32, 0},   // eRsLength
-    };
+    }};
 
     bool success = GetArgs(context, &args[0], args.size());
     if (!success)
@@ -1173,11 +1173,11 @@ RenderScriptRuntime::CaptureAllocationIn
         eRsForceZero
     };
 
-    std::array<ArgItem, 3> args = {
+    std::array<ArgItem, 3> args{{
         ArgItem{ArgItem::ePointer, 0}, // eRsContext
         ArgItem{ArgItem::ePointer, 0}, // eRsAlloc
         ArgItem{ArgItem::eBool, 0},    // eRsForceZero
-    };
+    }};
 
     bool success = GetArgs(context, &args[0], args.size());
     if (!success) // error case
@@ -1207,10 +1207,10 @@ RenderScriptRuntime::CaptureAllocationDe
         eRsAlloc,
     };
 
-    std::array<ArgItem, 2> args = {
+    std::array<ArgItem, 2> args{{
         ArgItem{ArgItem::ePointer, 0}, // eRsContext
         ArgItem{ArgItem::ePointer, 0}, // eRsAlloc
-    };
+    }};
 
     bool success = GetArgs(context, &args[0], args.size());
     if (!success)
@@ -1256,8 +1256,8 @@ RenderScriptRuntime::CaptureScriptInit(R
         eRsCachedDirPtr
     };
 
-    std::array<ArgItem, 4> args = {ArgItem{ArgItem::ePointer, 0}, ArgItem{ArgItem::ePointer, 0},
-                                   ArgItem{ArgItem::ePointer, 0}, ArgItem{ArgItem::ePointer, 0}};
+    std::array<ArgItem, 4> args{{ArgItem{ArgItem::ePointer, 0}, ArgItem{ArgItem::ePointer, 0},
+                                 ArgItem{ArgItem::ePointer, 0}, ArgItem{ArgItem::ePointer, 0}}};
     bool success = GetArgs(context, &args[0], args.size());
     if (!success)
     {

Modified: lldb/trunk/tools/driver/Driver.cpp
URL: http://llvm.org/viewvc/llvm-project/lldb/trunk/tools/driver/Driver.cpp?rev=260914&r1=260913&r2=260914&view=diff
==============================================================================
--- lldb/trunk/tools/driver/Driver.cpp (original)
+++ lldb/trunk/tools/driver/Driver.cpp Mon Feb 15 15:50:28 2016
@@ -914,7 +914,8 @@ PrepareCommandsForSourcing (const char *
         {
             fprintf(stderr, "error: write(%i, %p, %" PRIu64 ") failed (errno = %i) "
                             "when trying to open LLDB commands pipe\n",
-                    fds[WRITE], commands_data, static_cast<uint64_t>(commands_size), errno);
+                    fds[WRITE], static_cast<const void *>(commands_data),
+                    static_cast<uint64_t>(commands_size), errno);
         }
         else if (static_cast<size_t>(nrwr) == commands_size)
         {




More information about the lldb-commits mailing list