bugGNU Wget - Bugs: bug #50665, spelling fixes

 
 

bug #50665: spelling fixes

Submitter:  klemens <ka7>
Submitted:  Tue 28 Mar 2017 06:58:38 PM UTC
   
 
Category:  None Severity:  3 - Normal
Priority:  5 - Normal Status:  Fixed
Privacy:  Public Assigned to:  rockdaboot
Originator Name:  klemens Open/Closed:  Closed
Release:  None Operating System:  None
Reproducibility:  None Fixed Release:  None
Planned Release:  1.19.2 Regression:  None
Work Required:  None Patch Included:  Yes
* Mandatory Fields

Add a New Comment Rich Markup
   

Sat 01 Apr 2017 05:43:09 PM UTC, comment #2: 

Thanks, pushed.

Tim Ruehsen <rockdaboot>
Group administrator
Wed 29 Mar 2017 08:10:32 AM UTC, comment #1: 

Thanks, how cool is that tool !

Just some questions / remarks:

- do not touch words beginning with '--' as these could be options, e.g. '--dont-remove-listing' has been changed to '--don't-remove-listing'. That is why I can't simply apply your patch.

- What about the doc/ directory ? I can't believe there are no typos in the docs :-)

- Does the tool only check comments in source files ? Checking (printf) strings would also be interesting... but maybe it does and just didn't find any typos !?

- Wishlist: If the tool could git commit the changes with a detailed commit message (e.g. optionally GNU style) and create a patch with 'git format-patch -1', that would be amazing and a big step towards automatism.

Anyways, could you fix the first in the list and re-send a patch !?

Tim Ruehsen <rockdaboot>
Group administrator
Tue 28 Mar 2017 06:58:38 PM UTC, original submission:  

patch contains some spelling fixes ( just in comments ) as found by a bot ( https://github.com/ka7/misspell_fixer ). Any upcoming License changes (e.g. GPL2 to GPL3+) are hereby granted.


klemens <ka7>

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attach Files:
   
   
Comment:
   

Attached Files
file #40227:  0001-spelling-fixes.patch added by ka7 (13KiB - text/x-patch - v2 version (fixing the dont's ))
file #40183:  0001-spelling-fixes.patch added by ka7 (14KiB - text/x-patch - spelling fixes patch)

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by rockdaboot (Posted a comment)
  • -email is unavailable- added by ka7 (Submitted the item)
  •  

    Follow 7 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2017-04-01 rockdaboot StatusInspected Fixed
        Open/ClosedOpen Closed
        Planned ReleaseNone 1.19.2
    2017-04-01 ka7 Attached File- Added 0001-spelling-fixes.patch, #40227
    2017-03-29 rockdaboot StatusNone Inspected
        Assigned toNone rockdaboot
    2017-03-28 ka7 Attached File- Added 0001-spelling-fixes.patch, #40183

    Back to the top

    Powered by Savane 3.13-02a9.
    Corresponding source code