Hey Paul,

The change looks fine.

Minor formatting nit.  Could you adjust your bracing style to match the rest of 
the code, though?  You can either filter it through the clang formatter (there 
was a thread on how to do that within the last 2 - 3 weeks on lldb-dev or 
lldb-commits) or adjust the bits I list below.  The only bits I'm really 
noticing are related to else and curly brace starting position.

#1.

else {

would be:
else
{

#2:

} else
{

would be
}
else
{
}

Thanks!

http://reviews.llvm.org/D5232



_______________________________________________
lldb-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/lldb-commits

Reply via email to