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

Bug 315525

Summary: [f.p] False report for return if function is inside another function
Product: [Tools] CDT Reporter: Sergey Prigogin <eclipse.sprigogin>
Component: cdt-codanAssignee: Elena Laskavaia <elaskavaia.cdt>
Status: RESOLVED FIXED QA Contact: Elena Laskavaia <elaskavaia.cdt>
Severity: normal    
Priority: P3 CC: cdtdoug, yevshif
Version: 7.0   
Target Milestone: ---   
Hardware: PC   
OS: Linux   
Whiteboard:

Description Sergey Prigogin CLA 2010-06-02 22:18:42 EDT 
void test() {
  class A {
   public:
    int m() {
      return 0;
    }
  };
}

return 0; statement is flagged with: Return vas value, in function returning void.
Comment 1 Elena Laskavaia CLA 2010-06-02 22:52:52 EDT 
confused with function inside another one :O
Comment 2 CDT Genie CLA 2010-06-02 23:23:02 EDT 
*** cdt cvs genie on behalf of elaskavaia ***
Bug 315525: fix to suppress errors in inner functions

[*] ReturnChecker.java 1.10 http://dev.eclipse.org/viewcvs/index.cgi/org.eclipse.cdt/codan/org.eclipse.cdt.codan.checkers/src/org/eclipse/cdt/codan/internal/checkers/ReturnChecker.java?root=Tools_Project&r1=1.9&r2=1.10
Comment 3 Sergey Prigogin CLA 2010-06-02 23:34:48 EDT 
(In reply to comment #2)

With this fix would 

void test() {
  class A {
   public:
    void m() {
      return 0;
    }
  };
}

still produce an error?
Comment 4 Elena Laskavaia CLA 2010-06-03 09:34:43 EDT 
Nope. This is false negative now. I will fix it after 7.0
Comment 5 Elena Laskavaia CLA 2010-06-08 11:05:02 EDT 
was fixed in 7.0