[lldb-dev] [PATCH] Quick bugfix in source/Symbol/Symtab.cpp

Greg Clayton gclayton at apple.com
Tue Jul 20 11:30:34 PDT 2010


Committed revision 108871.

Thanks!

On Jul 19, 2010, at 7:05 PM, William Lynch wrote:

> It looks like the operator should be || not && in this case. 
> 
> Index: source/Symbol/Symtab.cpp
> ===================================================================
> --- source/Symbol/Symtab.cpp	(revision 108750)
> +++ source/Symbol/Symtab.cpp	(working copy)
> @@ -504,7 +504,7 @@
>  Symtab::CalculateSymbolSize (Symbol *symbol)
>  {
>      // Make sure this symbol is from this symbol table...
> -    if (symbol < m_symbols.data() && symbol >= m_symbols.data() + m_symbols.size())
> +    if (symbol < m_symbols.data() || symbol >= m_symbols.data() + m_symbols.size())
>          return 0;
>  
>      // See if this symbol already has a byte size?
> 
> _______________________________________________
> lldb-dev mailing list
> lldb-dev at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/lldb-dev





More information about the lldb-dev mailing list