[cdt-dev] Bug 376105 - "Add watchpoint (C/C++)" context menu option from

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [List Home]

[cdt-dev] Bug 376105 - "Add watchpoint (C/C++)" context menu option from memory view does not add a asterisk before the memory address value

From: Mike Wrighton <mike.wrighton@xxxxxxxxxxxxxx>
Date: Thu, 12 Apr 2012 17:13:50 -0500
Delivered-to: cdt-dev@xxxxxxxxxxx
List-archive: <https://dev.eclipse.org/mailman/private/cdt-dev>
List-help: <mailto:cdt-dev-request@eclipse.org?subject=help>
List-subscribe: <https://dev.eclipse.org/mailman/listinfo/cdt-dev>, <mailto:cdt-dev-request@eclipse.org?subject=subscribe>
List-unsubscribe: <https://dev.eclipse.org/mailman/options/cdt-dev>, <mailto:cdt-dev-request@eclipse.org?subject=unsubscribe>

Hi,

I've submitted a one-line patch for the bug described in https://bugs.eclipse.org/bugs/show_bug.cgi?id=376105 and I just wanted to get an opinion about whether this was a reasonable fix. I figured that since the code in AbstractToggleBreakpointAdapter.updateMemoryWatchpoint() is only called for the case of watchpoint addresses from the memory view, and the asterisk address prefix is standard C syntax (debugger-independent), this would be acceptable?

Thanks,

Mike

Prev by Date: Re: [cdt-dev] Debugging tutorial ?
Next by Date: [cdt-dev] Change ScannerConfigDiscoveryProfileId for an autotools project programmatically
Previous by thread: [cdt-dev] CDT Tag 8.0.2 updated?
Next by thread: [cdt-dev] Change ScannerConfigDiscoveryProfileId for an autotools project programmatically
Index(es):
- Date
- Thread

Breadcrumbs