Closed Bug 537398 Opened 15 years ago Closed 15 years ago

Fix copyright headers in libjar

Categories

(Core :: Networking: JAR, defect)

Product:
Core
Component:
Networking: JAR
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED FIXED

People

(Reporter: taras.mozilla, Assigned: taras.mozilla)

References

Details

Attachments

(1 file, 1 obsolete file)

rev2
2.18 KB, patch
reed
: review+
Details | Diff | Splinter Review 
Thanks to reed to pointing my mistake in https://bugzilla.mozilla.org/show_bug.cgi?id=536911#c12
Attached patch license fix (obsolete) — — Splinter Review 
I landed the other change in a rush because I wanted the fix to apply asap, but looks like I made a typo. Fixed in this patch.
Assignee: nobody → tglek
Attachment #419690 - Flags: review?(reed)
Component: Networking → Networking: JAR
OS: Linux → All
QA Contact: networking → networking.jar
Hardware: x86 → All
Comment on attachment 419690 [details] [diff] [review]
license fix

>+ * the Mozilla Foundation.

Drop the "the" on the two lines with this to just have "Mozilla Foundation".

As for modules/libjar/test/unit/test_corrupt_536911.js, you just need to add a missing period after "Foundation".
Attachment #419690 - Flags: review?(reed) → review-
Attached patch rev2 — — Splinter Review 

        Attachment #419690 -
        Attachment is obsolete: true

        Attachment #419694 -
        Flags: review?(reed)

    
    

    
  
Comment on attachment 419694 [details] [diff] [review]
rev2

r=reed

Thanks for fixing this so quickly.

        Attachment #419694 -
        Flags: review?(reed) → review+

    
    

    
  
http://hg.mozilla.org/mozilla-central/rev/599d0aa600c9
Status: NEW → RESOLVED
Closed: 15 years ago
Resolution: --- → FIXED

    
    

    
  
Is 1.9.2 affected by any of these problems? If so, should fix there, too. Since this is all in comments / test-only, no approval will be required.

    
    

    
  
yeah I'll land that together with bug 536911
Depends on: 536911

    
    

    
  
[It's not worth making a new patch, but as a matter of style and grammar, I prefer "the Mozilla Foundation" to "Mozilla Foundation".]

Gerv
Blocks: 536911
No longer depends on: 536911

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






  

    
  







  

    

      
Attachment

      

      
      
      
      
    

    

      

        

          

            
General

            

              Creator: 



            

            
Created: 

            
Updated: 

            
Size: