[lldb-dev] [PATCH] Ctor init order fix.
steve.wilson at start.ca
Tue Jul 6 10:10:23 PDT 2010
Unfortunately I originally responded to this mail using my work address,
which is not subscribed to the list. There will be a duplicate.
Greg Clayton <gclayton at apple.com> writes:
> For simple patches like this one, feel free to go ahead and just commit them!
I would if I had commit access! This is the first patch I have sent to
an LLVM related project.
> Greg Clayton
> On Jul 2, 2010, at 8:38 PM, Stephen Wilson wrote:
>> Hi all,
>> My hope is to help with the linux port. Initially I would like to clean
>> up many of the compiler warnings I am seeing as a way to familiarize
>> myself with the basic code layout. Attached patch fixes a ctor
>> initialization order warning.
>> I am seeing many warnings, largely trivial. Am I correct in assuming
>> small patches like this one (file-by-file basis) are preferred?
>> diff --git a/source/Symbol/LineEntry.cpp b/source/Symbol/LineEntry.cpp
>> index fb362b9..83c50f2 100644
>> --- a/source/Symbol/LineEntry.cpp
>> +++ b/source/Symbol/LineEntry.cpp
>> @@ -20,9 +20,9 @@ LineEntry::LineEntry() :
>> - is_terminal_entry(0),
>> - is_epilogue_begin(0)
>> + is_epilogue_begin(0),
>> + is_terminal_entry(0)
>> lldb-dev mailing list
>> lldb-dev at cs.uiuc.edu
More information about the lldb-dev