Uploaded image for project: 'WiredTiger'
  1. WiredTiger
  2. WT-7147

s_string not run on cpp files in codebase

    XMLWordPrintable

    Details

    • Type: Improvement
    • Status: Open
    • Priority: Major - P3
    • Resolution: Unresolved
    • Affects Version/s: None
    • Fix Version/s: Backlog
    • Component/s: None
    • Labels:

      Description

      The s_string utility checks for spelling errors in our source files, and is generally quite useful. Currently it doesn't check cxx files for spelling mistakes.

      A suggested change to fix this is:

      diff --git a/dist/s_string b/dist/s_string
      index 735490daf..2b55d86b7 100755
      --- a/dist/s_string
      +++ b/dist/s_string
      @@ -45,7 +45,7 @@ check() {
       # List of files to spellchk.
       l=`(cd .. &&
           find bench examples ext src test -name '*.[chsy]' &&
      -    find src -name '*.in')`
      +    find src -name '*.in' && find bench test -name '*.cxx')`
      

      There are two pieces of work here:
      1. Fix the script to it checks .cxx extension files.
      2. Fix the incorrectly spelled words

        Attachments

          Activity

            People

            Assignee:
            backlog-server-storage-engines Backlog - Storage Engines Team
            Reporter:
            luke.pearson Luke Pearson
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

              Dates

              Created:
              Updated: