Bug 163330 - CPPASTVisibilityLabel does not implement accept correctly
Summary: CPPASTVisibilityLabel does not implement accept correctly
Status: RESOLVED FIXED
Alias: None
Product: CDT
Classification: Tools
Component: cdt-core (show other bugs)
Version: 4.0   Edit
Hardware: All All
: P3 normal (vote)
Target Milestone: 4.0 M7   Edit
Assignee: Markus Schorn CLA
QA Contact:
URL:
Whiteboard:
Keywords: contributed
Depends on:
Blocks:
 
Reported: 2006-11-03 09:43 EST by Emanuel Graf CLA
Modified: 2008-06-20 11:26 EDT (History)
3 users (show)

See Also:

Attachments
Patch for CPPASTVisibilityLabel (1.06 KB, patch)
2006-11-03 09:45 EST, Emanuel Graf CLA 		no flags Details | Diff
New Patch with action.leave call (1.35 KB, patch)
2007-05-09 09:54 EDT, Emanuel Graf CLA 		bjorn.freeman-benson: iplog+
Details | Diff
Show Obsolete (1) 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 Emanuel Graf CLA 2006-11-03 09:43:29 EST 
The current implementation of accept() just returns true, without calling the visit-methode.
Comment 1 Emanuel Graf CLA 2006-11-03 09:45:16 EST 
Created attachment 53197 [details]
Patch for CPPASTVisibilityLabel
Comment 2 Emanuel Graf CLA 2007-05-09 09:54:36 EDT 
Created attachment 66465 [details]
New Patch with action.leave call

Could someone have a look at this patch please. This bug makes the visibility labels invisible for our ast re-writer.
Comment 3 Markus Schorn CLA 2007-05-09 11:15:01 EDT 
Thanks, I have applied the patch.
Comment 4 Doug Schaefer CLA 2008-06-03 15:03:43 EDT 
assigning
Comment 5 Doug Schaefer CLA 2008-06-03 15:04:05 EDT 
done