Closed Bug 305192 Opened 19 years ago Closed 19 years ago

nsMsgGroupView.cpp:448: warning: control reaches end of non-void function

Categories

(MailNews Core :: Backend, defect)

Product:
MailNews Core
Component:
Backend
Platform:
x86
Linux
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(Not tracked)

Status:
RESOLVED FIXED

People

(Reporter: wolfiR, Assigned: wolfiR)

Details

(Keywords: fixed1.8)

Attachments

(1 file)

return NS_OK if nothing to do
853 bytes, patch
Bienvenu
: review+
mscott
: approval1.8b4+
Details | Diff | Splinter Review 
nsresult nsMsgGroupView::HandleDayChange() doesn't return a value if the if-case
fails. I'm not really sure but the attached patch might be correct in that case?
Assignee: nobody → mozilla
Status: NEW → ASSIGNED

        Attachment #193148 -
        Flags: review?(bienvenu)

    
    

    
  
Comment on attachment 193148 [details] [diff] [review]
return NS_OK if nothing to do

yes, this should be fine. thx. I can check this in...

        Attachment #193148 -
        Flags: review?(bienvenu) → review+

    
    

    
  
fixed, thx.
Status: ASSIGNED → RESOLVED
Closed: 19 years ago
Resolution: --- → FIXED

    
  

        Attachment #193148 -
        Flags: approval1.8b4?

    
  

        Attachment #193148 -
        Flags: approval1.8b4? → approval1.8b4+

    
  
Keywords: fixed1.8
Product: Core → MailNews Core

          You need to log in
          before you can comment on or make changes to this bug.
        






  

    
  







  

    

      
Attachment

      

      
      
      
      
    

    

      

        

          

            
General

            

              Creator: 



            

            
Created: 

            
Updated: 

            
Size: