Closed Bug 425749 Opened 16 years ago Closed 5 years ago

Port |Bug 232488 – importMsgs.properties cleanup (remove unused strings + fix capitalization)| to TB

Categories

(MailNews Core :: Import, task)

Product:
MailNews Core
Component:
Import
Type:
task
Priority:
Not set
Severity:
trivial

Tracking

(Not tracked)

Status:
RESOLVED FIXED
Milestone:
mozilla1.9

People

(Reporter: sgautherie, Assigned: sgautherie)

References

Details

Attachments

(2 files, 1 obsolete file)

(Bv1-TB) <messenger/*> (Sync with SM)
4.37 KB, patch
Bienvenu
: review-
Details | Diff | Splinter Review
(Cv1-SM) <messenger/*> (Sync with TB)
6.90 KB, patch
neil
: review-
Details | Diff | Splinter Review 
I noticed this while working on bug 421405.
I'll attach a few more cleanups/synchronizations I noticed at the same time.
Assignee: nobody → sgautherie.bz
Status: NEW → ASSIGNED

        Attachment #312315 -
        Flags: review?(bienvenu)

    
    

    
  


  

        Attachment #312316 -
        Flags: review?(bienvenu)

    
    

    
  


  

        Attachment #312318 -
        Flags: superreview?(neil)

        Attachment #312318 -
        Flags: review?(neil)

    
  

        Attachment #312318 -
        Flags: superreview?(neil)

    
    

    
  
David, ping ?
Neil, ping ?

    
    

    
  
Comment on attachment 312318 [details] [diff] [review]
(Cv1-SM) <messenger/*> (Sync with TB)

See bug 375054 comment #8.

        Attachment #312318 -
        Flags: review?(neil) → review-

    
    

    
  
Comment on attachment 312316 [details] [diff] [review]
(Bv1-TB) <messenger/*> (Sync with SM)

I like the blank line - it makes it much more readable.

        Attachment #312316 -
        Flags: review?(bienvenu) → review-
Product: Core → MailNews Core

    
  

        Attachment #312315 -
        Attachment is obsolete: true

        Attachment #312315 -
        Flags: review?(bienvenu)

    
    

    
  
Comment on attachment 312315 [details] [diff] [review]
(Av1-TB) <importMsgs.properties> (Bug 232488 port to TB, ++)

sorry, this patch seems to be obsolete now.

    
    

    
  
What is the current status of this. Comment #7 indicates the patch for Tb is no longer needed and there has been no follow through on the SM Review- by Neil.

If the need for patches have been superseded, then how about this bug getting closed.

    
    

    
  
Thanks Ronald.

Let's declare this fixed


Status: ASSIGNED → RESOLVED
Type: defect → task
Closed: 5 years ago
Resolution: --- → FIXED

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






  

    
  







  

    

      
Attachment

      

      
      
      
      
    

    

      

        

          

            
General

            

              Creator: 



            

            
Created: 

            
Updated: 

            
Size: