[Lldb-commits] [PATCH] Mangled: Fix an 'unused variable' warning on GNU/Linux

[David Majnemer]

Hi zturner, tfiala,

Platforms which don't use LLDB's built-in demangler don't use the
'mangled_length' variable.  Instead, replace it's only use by an
expression it is equivalent to.

http://reviews.llvm.org/D4625

Files:
  source/Core/Mangled.cpp

Index: source/Core/Mangled.cpp
===================================================================
--- source/Core/Mangled.cpp
+++ source/Core/Mangled.cpp
@@ -5158,7 +5158,6 @@
 
         // Don't bother running anything that isn't mangled
         const char *mangled_cstr = m_mangled.GetCString();
-        long mangled_length = m_mangled.GetLength();
         if (cstring_is_mangled(mangled_cstr))
         {
             if (!m_mangled.GetMangledCounterpart(m_demangled))
@@ -5170,7 +5169,7 @@
                 // performance win, falling back to the full demangler only
                 // when necessary
                 char *demangled_name = FastDemangle (mangled_cstr,
-                                                     mangled_length);
+                                                     m_mangled.GetLength());
                 if (!demangled_name)
                     demangled_name = __cxa_demangle (mangled_cstr, NULL, NULL, NULL);
 #elif defined(_MSC_VER)
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D4625.11776.patch
Type: text/x-patch
Size: 977 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/lldb-commits/attachments/20140722/affcf10f/attachment.bin>