Style, white space & other mechanical cleanups

	* src/dbgengine/nmv-dbg-common.h (class OutOfBandRecord): Fix
	comments.
	(Output::ResultRecord::has_var_changes): Remove trailing white
	space.
	* src/dbgengine/nmv-gdb-engine.h: Flag the file as being a c++ one
	for emacs.
	* src/dbgengine/nmv-gdb-engine.cc
	(OnBreakpointHandler::{has_overloads_prompt,
	extract_overloads_choice_prompt_values, has_breakpoints_set}):
	Change member function declaration style.
	(GDBEngine::disable_breakpoint): Remove unused variable.
	(GDBEngine::append_breakpoint_to_cache): Wrap too long line.
	Remove useless white space.
	(GDBEngine::append_breakpoint_to_cache): Likewise.
	* src/dbgengine/nmv-gdbmi-parser.cc (LOG_PARSING_ERROR): Remove
	this unused macro.  Rename LOG_PARSING_ERROR2 into
	LOG_PARSING_ERROR.
	(LOG_PARSING_ERROR_MSG): Remove this unused macro.  Rename
	LOG_PARSING_ERROR_MSG2 into LOG_PARSING_ERROR_MSG.
	(CHECK_END): Remove this unused macro.  Rename CHECK_END2 into
	this.
	(CHECK_END_BREAK): Remove this unused macro.
	(SKIP_WS): Remove.  Renamed SKIP_WS2 into this.
	(SKIP_BLANK): Remove.  Renamed SKIP_BLANK2 into this.
	(lots of functions): Update for the renaming above.
	(GDBMIParser::{parse_breakpoint, parse_threads_list): Remove
	trailing white space.
	* src/persp/dbgperspective/nmv-breakpoints-view.cc
	(BreakpointsView::Priv::should_process_now): Remove useless white
	space.
	* src/persp/dbgperspective/nmv-dbg-perspective.cc
	(DBGPerspective::execute_session, DBGPerspective::get_breakpoint):
	White space fixes.
Rename IDebugger::Breakpoint::number to IDebugger::Breakpoint::id

	* src/dbgengine/nmv-i-debugger.h (IDebugger::Breakpoint::id):
	Rename IDebugger::Breakpoint::number into this.  What we are
	getting is a string id, not a an integer number (the id can be
	"2.3", for instance).
	(IDebugger::Breakpoint::number): New method that always return the
	sub breakpoint number.
	* src/dbgengine/nmv-gdb-engine.cc
	(OnBreakpointHandler::do_handle)
	(GDBEngine::append_breakpoint_to_cache, ): Update for the
	Breakpoint::id rename.
	* src/dbgengine/nmv-gdbmi-parser.cc
	(GDBMIParser::parse_result_record): Likewise.
	* src/persp/dbgperspective/nmv-breakpoints-view.cc
	(BreakpointsView::Priv::{breakpoint_list_has_id,
	find_breakpoint_in_model, update_breakpoint,
	on_debugger_breakpoints_list_signal}): Likewise.
	* src/persp/dbgperspective/nmv-dbg-perspective.cc
	(DBGPerspective::{on_breakpoint_delete_action,
	on_debugger_breakpoints_list_signal,
	do_jump_and_break_to_location, re_initialize_set_breakpoints,
	append_breakpoint_to_cache, delete_breakpoint, toggle_countpoint,
	toggle_breakpoint_enabled}): Likewise.
	* tests/test-gdbmi.cc (test_breakpoint_table, test_breakpoint):
	Likewise.
Handle deleting all sub-breakpoints at once

	* src/dbgengine/nmv-i-debugger.h
	(IDebugger::Breakpoint::parent_id): New inline method.
	* src/dbgengine/nmv-gdb-engine.cc (GDBEngine::delete_breakpoint):
	When passed the id of a sub-breakpoint, delete its parent
	breakpoint.
	* src/persp/dbgperspective/nmv-breakpoints-view.cc
	(BreakpointsView::Priv::on_debugger_breakpoint_deleted_signal):
	Delete all sub-breakpoint which parent_id equals the id of the
	deleted breakpoint.
	* src/persp/dbgperspective/nmv-dbg-perspective.cc
	(DBGPerspective::on_debugger_breakpoint_deleted_signal): Likewise.
	(DBGPerspective::record_and_save_session): Avoid recording the
	same breakpoint twice b/c we'd recorded to sub-breakpoint of the
	same parent breakpoint.
	(DBGPerspective::execute_program): Do not save/re-set a breakpoint
	twice b/c we'd have saved two sub-breakpoints with the same parent
	id.
	(DBGPerspective::delete_breakpoint): Walk all the breakpoints and
	delete all breakpoints which file location equals the ones of the
	breakpoint to delete.
Support modified-breakpoint async output from GDB

	* src/dbgengine/nmv-dbg-common.h
	(Output::OutOfBandRecord::{m_has_modified_breakpoint,
	m_modified_breakpoint): New members.
	(Output::OutOfBandRecord::{has_modified_breakpoint,
	modified_breakpoint}): New accessors.
	(Output::OutOfBandRecord::clear): Clear m_has_modified_breakpoint
	and m_modified_breakpoint.
	(Output::OutOfBandRecords): New typedef.
	* src/dbgengine/nmv-gdbmi-parser.h
	(GDBMIParser::parse_breakpoint_modified_async_output): New
	declaration.
	* src/dbgengine/nmv-gdbmi-parser.cc (CHECK_END): Re-use
	END_OF_INPUT for this macro.
	(PARSING_ERROR_IF_END): New macro.
	(PREFIX_*, NDELETED, NUMCHILD, CHANGELIST, PATH_EXPR): Make these
	global variables static.
	(PREFIX_BREAKPOINT_MODIFIED_ASYNC_OUTPUT): New static global
	variable.
	(GDBMIParser::parse_breakpoint_modified_async_output): New function.
	(GDBMIParser::parse_out_of_band_record):  New the new function
	above.
	* src/dbgengine/nmv-i-debugger.h (OverloadsChoiceEntry): New
	typedef.
	* src/dbgengine/nmv-gdb-engine.cc
	(OnBreakpointHandler::{has_modified_breakpoint,
	notify_breakpoint_deleted_signal,
	append_bp_to_cache_and_notify_bp_set): New helper functions.
	(OnBreakpointHandler::can_handle): If the output contains an
	out-of-band-record with the modified breakpoint async output then
	the OnBreakpointHandler can handle this output.
	(OnBreakpointHandler::do_handle): New the above. Detect if a
	breakpoint was modified.  If so, delete its previous version from
	the cache, notify listeners about the deletion, add the new
	version to the cache and notify listeners about the addition.
	Also, rename the has_breaks boolean into has_breaks_set.  Remove
	some trailing white space.  Use the new
	notify_breakpoint_deleted_signal to delete some old code, making
	this a bit more maintainable.
	* tests/test-gdbmi.cc (gv_breakpoint_modified_async_output0): New
	global with a modified-breakpoint async output from GDB.
	(test_breakpoint): Add testing about parsing the
	modified-breakpoint async output from GDB above.