Order of initialization lists.

This patch fixes all of the warnings due to unordered initialization lists.

Patch by Marco Minutoli.

llvm-svn: 129290
This commit is contained in:
Stephen Wilson
2011-04-11 19:41:40 +00:00
parent 2b899767e1
commit 71c21d18c3
22 changed files with 70 additions and 69 deletions

View File

@@ -70,9 +70,9 @@ private:
uint8_t ptr_encoding;
lldb_private::UnwindPlan::Row initial_row;
CIE(dw_offset_t offset) : cie_offset(offset), initial_row(), version (-1),
code_align (0), data_align (0), return_addr_reg_num (-1),
inst_offset (0), inst_length (0), ptr_encoding (0) {}
CIE(dw_offset_t offset) : cie_offset(offset), version (-1), code_align (0),
data_align (0), return_addr_reg_num (-1), inst_offset (0),
inst_length (0), ptr_encoding (0), initial_row() {}
};
typedef lldb::SharedPtr<CIE>::Type CIESP;
@@ -82,7 +82,7 @@ private:
AddressRange bounds; // function bounds
dw_offset_t offset; // offset to this FDE within the Section
FDEEntry () : offset (0), bounds () { }
FDEEntry () : bounds (), offset (0) { }
inline bool
operator<(const DWARFCallFrameInfo::FDEEntry& b) const

View File

@@ -80,8 +80,8 @@ public:
//----------------------------------------------------------------------
CleanUp (value_type value, value_type invalid, CallbackType callback) :
m_current_value (value),
m_callback (callback),
m_invalid_value (invalid),
m_callback (callback),
m_callback_called (false),
m_invalid_value_is_valid (true)
{

View File

@@ -69,8 +69,8 @@ EmulateInstruction::EmulateInstruction
m_write_mem_callback (write_mem_callback),
m_read_reg_callback (read_reg_callback),
m_write_reg_callback (write_reg_callback),
m_opcode (),
m_opcode_pc (LLDB_INVALID_ADDRESS),
m_opcode (),
m_advance_pc (false)
{
}

View File

@@ -48,6 +48,7 @@ static lldb::user_id_t g_value_obj_uid = 0;
ValueObject::ValueObject (ValueObject &parent) :
UserID (++g_value_obj_uid), // Unique identifier for every value object
m_parent (&parent),
m_update_point (parent.GetUpdatePoint ()),
m_name (),
m_data (),
m_value (),
@@ -66,8 +67,7 @@ ValueObject::ValueObject (ValueObject &parent) :
m_children_count_valid (false),
m_old_value_valid (false),
m_pointers_point_to_load_addrs (false),
m_is_deref_of_parent (false),
m_update_point (parent.GetUpdatePoint ())
m_is_deref_of_parent (false)
{
}
@@ -77,6 +77,7 @@ ValueObject::ValueObject (ValueObject &parent) :
ValueObject::ValueObject (ExecutionContextScope *exe_scope) :
UserID (++g_value_obj_uid), // Unique identifier for every value object
m_parent (NULL),
m_update_point (exe_scope),
m_name (),
m_data (),
m_value (),
@@ -95,8 +96,7 @@ ValueObject::ValueObject (ExecutionContextScope *exe_scope) :
m_children_count_valid (false),
m_old_value_valid (false),
m_pointers_point_to_load_addrs (false),
m_is_deref_of_parent (false),
m_update_point (exe_scope)
m_is_deref_of_parent (false)
{
}
@@ -1410,16 +1410,17 @@ ValueObject::AddressOf (Error &error)
}
ValueObject::EvaluationPoint::EvaluationPoint () :
m_stop_id (0),
m_thread_id (LLDB_INVALID_UID)
m_thread_id (LLDB_INVALID_UID),
m_stop_id (0)
{
}
ValueObject::EvaluationPoint::EvaluationPoint (ExecutionContextScope *exe_scope, bool use_selected):
m_stop_id (0),
m_thread_id (LLDB_INVALID_UID),
m_needs_update (true),
m_first_update (true)
m_first_update (true),
m_thread_id (LLDB_INVALID_UID),
m_stop_id (0)
{
ExecutionContext exe_ctx;
ExecutionContextScope *computed_exe_scope = exe_scope; // If use_selected is true, we may find a better scope,
@@ -1477,12 +1478,12 @@ ValueObject::EvaluationPoint::EvaluationPoint (ExecutionContextScope *exe_scope,
ValueObject::EvaluationPoint::EvaluationPoint (const ValueObject::EvaluationPoint &rhs) :
m_exe_scope (rhs.m_exe_scope),
m_needs_update(true),
m_first_update(true),
m_target_sp (rhs.m_target_sp),
m_process_sp (rhs.m_process_sp),
m_thread_id (rhs.m_thread_id),
m_stack_id (rhs.m_stack_id),
m_needs_update(true),
m_first_update(true),
m_stop_id (0)
{
}

View File

@@ -34,8 +34,8 @@ ASTStructExtractor::ASTStructExtractor(ASTConsumer *passthrough,
m_passthrough_sema (NULL),
m_sema (NULL),
m_action (NULL),
m_struct_name (struct_name),
m_function (function)
m_function (function),
m_struct_name (struct_name)
{
if (!m_passthrough)
return;

View File

@@ -50,9 +50,9 @@ using namespace clang;
ClangExpressionDeclMap::ClangExpressionDeclMap (bool keep_result_in_memory) :
m_found_entities (),
m_struct_members (),
m_keep_result_in_memory (keep_result_in_memory),
m_parser_vars (),
m_struct_vars (),
m_keep_result_in_memory (keep_result_in_memory)
m_struct_vars ()
{
EnableStructVars();
}

View File

@@ -28,16 +28,16 @@ using namespace clang;
ClangExpressionVariable::ClangExpressionVariable(ExecutionContextScope *exe_scope, lldb::ByteOrder byte_order, uint32_t addr_byte_size) :
m_parser_vars(),
m_jit_vars (),
m_frozen_sp (new ValueObjectConstResult(exe_scope, byte_order, addr_byte_size)),
m_flags (EVNone)
m_flags (EVNone),
m_frozen_sp (new ValueObjectConstResult(exe_scope, byte_order, addr_byte_size))
{
}
ClangExpressionVariable::ClangExpressionVariable (const lldb::ValueObjectSP &valobj_sp) :
m_parser_vars(),
m_jit_vars (),
m_frozen_sp (valobj_sp),
m_flags (EVNone)
m_flags (EVNone),
m_frozen_sp (valobj_sp)
{
}

View File

@@ -45,11 +45,11 @@ ClangUserExpression::ClangUserExpression (const char *expr,
m_expr_text (expr),
m_expr_prefix (expr_prefix ? expr_prefix : ""),
m_transformed_text (),
m_desired_type (NULL, NULL),
m_cplusplus (false),
m_objectivec (false),
m_needs_object_ptr (false),
m_const_object (false),
m_desired_type (NULL, NULL)
m_const_object (false)
{
}

View File

@@ -462,8 +462,8 @@ private:
IRDynamicChecks::IRDynamicChecks(DynamicCheckerFunctions &checker_functions,
const char *func_name) :
ModulePass(ID),
m_checker_functions(checker_functions),
m_func_name(func_name)
m_func_name(func_name),
m_checker_functions(checker_functions)
{
}

View File

@@ -38,15 +38,15 @@ IRForTarget::IRForTarget (lldb_private::ClangExpressionDeclMap *decl_map,
lldb_private::Stream *error_stream,
const char *func_name) :
ModulePass(ID),
m_resolve_vars(resolve_vars),
m_func_name(func_name),
m_decl_map(decl_map),
m_CFStringCreateWithBytes(NULL),
m_sel_registerName(NULL),
m_func_name(func_name),
m_resolve_vars(resolve_vars),
m_const_result(const_result),
m_error_stream(error_stream),
m_has_side_effects(false),
m_result_is_pointer(false),
m_error_stream(error_stream)
m_result_is_pointer(false)
{
}

View File

@@ -31,10 +31,10 @@ IRToDWARF::IRToDWARF(lldb_private::ClangExpressionVariableList &local_vars,
lldb_private::StreamString &strm,
const char *func_name) :
ModulePass(ID),
m_func_name(func_name),
m_local_vars(local_vars),
m_decl_map(decl_map),
m_strm(strm),
m_func_name(func_name)
m_strm(strm)
{
}

View File

@@ -40,20 +40,20 @@ RegisterContextLLDB::RegisterContextLLDB
) :
RegisterContext (thread, frame_number),
m_thread(thread),
m_next_frame(next_frame),
m_sym_ctx(sym_ctx),
m_all_registers_available(false),
m_registers(),
m_cfa (LLDB_INVALID_ADDRESS),
m_next_frame(next_frame),
m_fast_unwind_plan_sp (),
m_full_unwind_plan_sp (),
m_all_registers_available(false),
m_frame_type (-1),
m_cfa (LLDB_INVALID_ADDRESS),
m_start_pc (),
m_current_pc (),
m_frame_number (frame_number),
m_full_unwind_plan_sp (),
m_fast_unwind_plan_sp (),
m_frame_type (-1),
m_current_offset (0),
m_current_offset_backed_up_one (0),
m_sym_ctx_valid (false)
m_current_offset_backed_up_one (0),
m_sym_ctx(sym_ctx),
m_sym_ctx_valid (false),
m_frame_number (frame_number),
m_registers()
{
m_sym_ctx.Clear();
m_sym_ctx_valid = false;

View File

@@ -167,10 +167,10 @@ private:
};
AssemblyParse_x86::AssemblyParse_x86 (Target& target, Thread* thread, int cpu, AddressRange func) :
m_target (target), m_thread (thread), m_cpu(cpu), m_func_bounds(func),
m_target (target), m_thread (thread), m_func_bounds(func), m_cur_insn (),
m_machine_ip_regnum (-1), m_machine_sp_regnum (-1), m_machine_fp_regnum (-1),
m_lldb_ip_regnum (-1), m_lldb_sp_regnum (-1), m_lldb_fp_regnum (-1),
m_wordsize (-1), m_cur_insn ()
m_wordsize (-1), m_cpu(cpu)
{
int *initialized_flag = NULL;
m_lldb_ip_regnum = m_lldb_sp_regnum = m_lldb_fp_regnum = -1;

View File

@@ -30,13 +30,13 @@ DWARFCallFrameInfo::DWARFCallFrameInfo(ObjectFile& objfile, SectionSP& section,
m_objfile (objfile),
m_section (section),
m_reg_kind (reg_kind), // The flavor of registers that the CFI data uses (enum RegisterKind)
m_flags (),
m_cie_map (),
m_cfi_data (),
m_cfi_data_initialized (false),
m_fde_index (),
m_fde_index_initialized (false),
m_is_eh_frame (is_eh_frame),
m_flags ()
m_is_eh_frame (is_eh_frame)
{
}

View File

@@ -20,8 +20,8 @@ using namespace lldb_private;
Symbol::Symbol() :
SymbolContextScope (),
UserID (),
SymbolContextScope (),
m_mangled (),
m_type (eSymbolTypeInvalid),
m_type_data (0),
@@ -53,8 +53,8 @@ Symbol::Symbol
uint32_t size,
uint32_t flags
) :
SymbolContextScope (),
UserID (symID),
SymbolContextScope (),
m_mangled (name, name_is_mangled),
m_type (type),
m_type_data (0),
@@ -84,8 +84,8 @@ Symbol::Symbol
const AddressRange &range,
uint32_t flags
) :
SymbolContextScope (),
UserID (symID),
SymbolContextScope (),
m_mangled (name, name_is_mangled),
m_type (type),
m_type_data (0),
@@ -103,8 +103,8 @@ Symbol::Symbol
}
Symbol::Symbol(const Symbol& rhs):
SymbolContextScope (rhs),
UserID (rhs),
SymbolContextScope (rhs),
m_mangled (rhs.m_mangled),
m_type (rhs.m_type),
m_type_data (rhs.m_type_data),

View File

@@ -63,8 +63,8 @@ Type::Type () :
m_symbol_file (NULL),
m_context (NULL),
m_encoding_type (NULL),
m_encoding_uid_type (eEncodingInvalid),
m_encoding_uid (0),
m_encoding_uid_type (eEncodingInvalid),
m_byte_size (0),
m_decl (),
m_clang_type (NULL),
@@ -79,8 +79,8 @@ Type::Type (const Type &rhs) :
m_symbol_file (rhs.m_symbol_file),
m_context (rhs.m_context),
m_encoding_type (rhs.m_encoding_type),
m_encoding_uid_type (rhs.m_encoding_uid_type),
m_encoding_uid (rhs.m_encoding_uid),
m_encoding_uid_type (rhs.m_encoding_uid_type),
m_byte_size (rhs.m_byte_size),
m_decl (rhs.m_decl),
m_clang_type (rhs.m_clang_type),

View File

@@ -33,8 +33,8 @@ UnwindTable::UnwindTable (ObjectFile& objfile) :
m_object_file (objfile),
m_unwinds (),
m_initialized (false),
m_eh_frame (NULL),
m_assembly_profiler (NULL)
m_assembly_profiler (NULL),
m_eh_frame (NULL)
{
}

View File

@@ -38,10 +38,10 @@ StackFrameList::StackFrameList
) :
m_thread (thread),
m_prev_frames_sp (prev_frames_sp),
m_show_inlined_frames (show_inline_frames),
m_mutex (Mutex::eMutexTypeRecursive),
m_frames (),
m_selected_frame_idx (0)
m_selected_frame_idx (0),
m_show_inlined_frames (show_inline_frames)
{
}

View File

@@ -1174,10 +1174,10 @@ Target::RunStopHooks ()
Target::StopHook::StopHook (lldb::TargetSP target_sp, lldb::user_id_t uid) :
UserID (uid),
m_target_sp (target_sp),
m_active (true),
m_commands (),
m_specifier_sp (),
m_thread_spec_ap(NULL)
m_thread_spec_ap(NULL),
m_active (true)
{
}
@@ -1186,8 +1186,8 @@ Target::StopHook::StopHook (const StopHook &rhs) :
m_target_sp (rhs.m_target_sp),
m_commands (rhs.m_commands),
m_specifier_sp (rhs.m_specifier_sp),
m_active (rhs.m_active),
m_thread_spec_ap (NULL)
m_thread_spec_ap (NULL),
m_active (rhs.m_active)
{
if (rhs.m_thread_spec_ap.get() != NULL)
m_thread_spec_ap.reset (new ThreadSpec(*rhs.m_thread_spec_ap.get()));

View File

@@ -43,8 +43,8 @@ ThreadPlanTestCondition::ThreadPlanTestCondition (
lldb::BreakpointLocationSP break_loc_sp,
bool stop_others) :
ThreadPlan (ThreadPlan::eKindTestCondition, "test condition", thread, eVoteNoOpinion, eVoteNoOpinion),
m_exe_ctx (exe_ctx),
m_expression (expression),
m_exe_ctx (exe_ctx),
m_break_loc_sp (break_loc_sp),
m_did_stop (false),
m_stop_others (stop_others)

View File

@@ -33,17 +33,17 @@ using namespace lldb_private;
#pragma mark ThreadPlanTracer
ThreadPlanTracer::ThreadPlanTracer (Thread &thread, lldb::StreamSP &stream_sp) :
m_thread (thread),
m_single_step(true),
m_enabled (false),
m_thread (thread),
m_stream_sp (stream_sp)
{
}
ThreadPlanTracer::ThreadPlanTracer (Thread &thread) :
m_thread (thread),
m_single_step(true),
m_enabled (false),
m_thread (thread),
m_stream_sp ()
{
}

View File

@@ -342,8 +342,8 @@ Driver::OptionData::OptionData () :
m_debug_mode (false),
m_print_version (false),
m_print_help (false),
m_seen_options(),
m_use_external_editor(false)
m_use_external_editor(false),
m_seen_options()
{
}