Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 326965

Summary: Enhancement request: Add a hot key for the Find/Replace function for the memory views
Product: [Tools] CDT Reporter: Andre St. Laurent <st.laurent>
Component: cdt-memoryAssignee: Randy Rohrbach <Randy.Rohrbach>
Status: RESOLVED FIXED QA Contact: Ted Williams <ted>
Severity: enhancement    
Priority: P3 CC: pawel.1.piech, pwebster, Randy.Rohrbach, remy.suen
Version: 8.0Flags: Randy.Rohrbach: review? (pawel.1.piech)
Target Milestone: 8.0   
Hardware: PC   
OS: Windows XP   
Whiteboard:
Attachments:
Description Flags
Patch for CTRL/F to bring up the Find/Replace dialog box for the Memory and Memory Browser Views
none
Standard copyright added
none
Update patch to put more checking in and use the same property set. cdtdoug: iplog-

Description Andre St. Laurent CLA 2010-10-04 15:23:32 EDT 
Build Identifier: 

Currently, in order to cause the "Find/Replace Memory" dialog to appear you must use the Memory or Memory Browser view "Find/Replace ..." pull-down menu item.

This request proposes adding a hot key/binding to the "Find/Replace" function, with the most obvious choice being CTRL/F.

Reproducible: Always

Steps to Reproduce:
1. Create and run the project of your choice.
2. Add the Memory or Memory Browser view if not already displayed
3. Use the view pull-down menu to display the "Find/Replace Memory" dialog box; dismiss it.
4. Note that there are no keyboard accelerators to invoke the dialog box.
Comment 1 Pawel Piech CLA 2010-10-05 11:18:10 EDT 
Reassigning to CDT.  The Find action is contributed by the org.eclipse.cdt.debug.ui.memorysearch plugin.
Comment 2 Andre St. Laurent CLA 2010-10-06 08:20:37 EDT 
Created attachment 180320 [details]
Patch for CTRL/F to bring up the Find/Replace dialog box for the Memory and Memory Browser Views

This patch adds the ability for users to quickly pop up the Find/Replace dialog box using CTRL/F when focus is set to the Memory or Memory Browser views.
Comment 3 Pawel Piech CLA 2010-10-06 09:07:33 EDT 
Hi Andre,
Could you add the standard copyright notice to the new file:

/*******************************************************************************
 * Copyright (c) 2010 Wind River Systems and others.
 * All rights reserved. This program and the accompanying materials
 * are made available under the terms of the Eclipse Public License v1.0
 * which accompanies this distribution, and is available at
 * http://www.eclipse.org/legal/epl-v10.html
 *
 * Contributors:
 *     Wind River Systems - initial API and implementation
 *******************************************************************************/
Comment 4 Andre St. Laurent CLA 2010-10-06 10:01:30 EDT 
Created attachment 180331 [details]
Standard copyright added

Standard copyright added.
Comment 5 Randy Rohrbach CLA 2010-10-18 15:23:49 EDT 
I'm taking this one.

Randy
Comment 6 Randy Rohrbach CLA 2010-10-18 15:25:33 EDT 
Created attachment 181124 [details]
Update patch to put more checking in and use the same property set.

Update patch to put more checking in and use the same property set. This makes it so no matter how you bring up the dialog it shows the same content.
Comment 7 Randy Rohrbach CLA 2010-10-18 16:22:54 EDT 
Pawel

   I committed the fix to HEAD for CDT 8.0. Please review the changes.

   Thanks

Randy