Bug 181395 - [PATCH] RpmMacroProposalsList replaceAll fix
Summary: [PATCH] RpmMacroProposalsList replaceAll fix
Status: RESOLVED FIXED
Alias: None
Product: Linux Tools
Classification: Tools
Component: RPM (show other bugs)
Version: unspecified   Edit
Hardware: PC Linux
: P3 normal (vote)
Target Milestone: ---   Edit
Assignee: RPM Inbox CLA
QA Contact:
URL:
Whiteboard:
Keywords: contributed
Depends on:
Blocks:
 
Reported: 2007-04-06 07:31 EDT by Alphonse Van Assche CLA
Modified: 2009-01-23 10:21 EST (History)
1 user (show)

See Also:

Attachments
fix replaceAll() another replaceAll() bug in RpmMacroProposalsList (803 bytes, patch)
2007-04-06 07:43 EDT, Alphonse Van Assche CLA 		overholt: iplog+
Details | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Alphonse Van Assche CLA 2007-04-06 07:31:19 EDT 
There is a other bug in RpmMacroProposalsList, I have fixed it in the same way that Andrew have used yesterday (commit 517)

Have a nice day,
Alphonse
Comment 1 Alphonse Van Assche CLA 2007-04-06 07:43:53 EDT 
Created attachment 63165 [details]
fix replaceAll() another replaceAll() bug in RpmMacroProposalsList

Just for the record, Have found this bug while writing new test case for the folding, hover, completion functionality, now test case usefulness look crystal-clear for me.
Comment 2 Andrew Overholt CLA 2007-04-06 19:28:36 EDT 
Committed as rev. 526.  Thanks, Alphonse!

2007-04-06  Alphonse Van Assche  <alcapcom@gmail.com>

	* src/org/eclipse/linuxtools/rpm/ui/editor/RpmMacroProposalsList.java
	(getValue): Escape "?" character in regex. Bug #181395.
Comment 3 Andrew Overholt CLA 2009-01-23 10:21:59 EST 
Comment on attachment 63165 [details]
fix replaceAll() another replaceAll() bug in RpmMacroProposalsList

Setting iplog flag.