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