Opened 8 years ago
Closed 5 years ago
#6230 closed task (wontfix)
Fix spelling as much as possible
Reported by: | Kurt Schwehr | Owned by: | Kurt Schwehr |
---|---|---|---|
Priority: | normal | Milestone: | closed_because_of_github_migration |
Component: | default | Version: | unspecified |
Severity: | normal | Keywords: | |
Cc: | k.dejong@… |
Description
This ticket is mostly to describe how to find spelling mistakes. The simplest way for me to is in emacs to highlight a region and use M-$ ( or M-x ispell-region). Lots of other editors have ways to spell check code.
To get a list of the spelling errors possibly fixed in a change, you can start with this:
e.g.
acceptad accerated acces accessable accessor accessors accf accidently accomodate accomodation accompagnying accompaying accout accres accross accum accumualte
Change History (9)
comment:1 by , 8 years ago
follow-up: 6 comment:4 by , 8 years ago
Cc: | added |
---|
comment:5 by , 8 years ago
rouault, Thanks for upstreaming the lib*tiff stuff. There are also fixes in zlib, libqhull, degrib, hdf-eos, pcidsk, mitab that should be upstreamed. And there will be more. I was going to power through the rest of the list of possible spelling fixes, and was thinking that it would be good to batch those all up for upstreams. I'm sure some fixes will get lost in updates, but this is nice big just forward even if not perfect.
I looked at swig and the head of master for swig no longer has the "aquires" typos.
I am avoiding most of the actual code changes from spelling right now. And there are far fewer mispelling in variable, function and method names.
Oh... and another handy tr command:
comment:6 by , 8 years ago
Replying to rouault:
Kor, r31724 contains typo fixes for libcsf. What is the procedure to submit changes for upstream libcsf ? Hum I see we have also pushed a few libcsf changes in r31434 (other typo fixes) and r31640 ( -Wshorten-64-to-32 related warning fixes)
Hi Even, Thanks for letting me know. I will make sure the updates are merged into upstream libcsf.
comment:7 by , 8 years ago
Do note that the way I'm looking for spelling errors fails to find them at the beginning of lines.
- r31710 - misc
- r31724 - "a"
- r31729 - "b"
- r31749 - "c"
- r31750 - dictionnary
- r31793 - "d" + r31794 and r31800 to fixup mistake with source:trunk/autotest/alg/warp.py
- r31812 - "e" + r31813 to fix gdalcompare.py
- r31826 - "f" + r31828 to undo mistake in ogr_libkml.py
- r31852 - "g"
- r31853 - "h"
- r31955 - "i"
- r31958 - "j"
- r31959 - "k"
- r31961 - "l" (L)
- r31978 - "m"
- r31984 - "n"
- r32002 - "o"
- r32058 - "p"
- r32059 - "q"
- r32067 - "r" and r32068 fix of recognized
- r32092 - "s"
- r32099 - "t"
- r32100 - "u"
- r32104 - "v"
- r32105 - "w"
- None - "x"
- r32106 - "y"
- r32107 - "z"
comment:8 by , 8 years ago
Next tasks
- Make a search of the words already fixed to see where they still exist (my previous searches were not perfect)
- Upstream changes to code bases that are included in the GDAL tree (e.g. degrib, dgn, giflib, hdf-eos, libcsf, libgeotif, libjpeg, libjson, libqhull, libtif, pcidsk, zip)
comment:9 by , 5 years ago
Milestone: | → closed_because_of_github_migration |
---|---|
Resolution: | → wontfix |
Status: | new → closed |
This ticket has been automatically closed because Trac is no longer used for GDAL bug tracking, since the project has migrated to GitHub. If you believe this ticket is still valid, you may file it to https://github.com/OSGeo/gdal/issues if it is not already reported there.
r31710