You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@thrift.apache.org by "Jens Geyer (JIRA)" <ji...@apache.org> on 2014/11/23 21:55:12 UTC

[jira] [Comment Edited] (THRIFT-2849) Spell errors reported by codespell tool

    [ https://issues.apache.org/jira/browse/THRIFT-2849?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14222494#comment-14222494 ] 

Jens Geyer edited comment on THRIFT-2849 at 11/23/14 8:54 PM:
--------------------------------------------------------------

Hi [~anatolikus],

if you have a patch at hand, we like to review it. 
[More info about how to contribute can be found here|http://thrift.apache.org/docs/HowToContribute].




was (Author: jensg):
Hi @Anatol Pomozov,

if you have a patch at hand, we like to review it. 
[More info about how to contribute can be found here|http://thrift.apache.org/docs/HowToContribute].



> Spell errors reported by codespell tool
> ---------------------------------------
>
>                 Key: THRIFT-2849
>                 URL: https://issues.apache.org/jira/browse/THRIFT-2849
>             Project: Thrift
>          Issue Type: Bug
>          Components: Compiler (General)
>            Reporter: Anatol Pomozov
>            Priority: Trivial
>
> Codespell is a useful tool that find (and optionally fixes) common spell errors. It is what it found for current HEAD:
> [anatol@arch thrift]$ codespell 
> ./CHANGES:9: Extention  ==> Extension
> ./CHANGES:270: compatiblity  ==> compatibility
> ./CHANGES:560: libary  ==> library
> ./CHANGES:562: reproducable  ==> reproducible
> ./debian/rules:196: independant  ==> independent
> ./debian/rules:200: dependant  ==> dependent
> ./contrib/parse_profiling.py:45: funtion  ==> function
> ./contrib/vagrant/centos-6.5/README.md:18: preperation  ==> preparation
> ./contrib/fb303/TClientInfo.h:195: additonal  ==> additional
> ./contrib/fb303/aclocal/ax_javac_and_java.m4:10: currenly  ==> currently
> ./contrib/thrift-maven-plugin/src/main/java/org/apache/thrift/maven/AbstractThriftMojo.java:196: occured  ==> occurred
> ./contrib/zeromq/csharp/TZmqClient.cs:50: reponse  ==> response
> ./test/README.md:12: acording  ==> according
> ./test/DocTest.thrift:158: cannonical  ==> canonical
> ./test/ThriftTest.thrift:175: seperated  ==> separated
> ./test/ThriftTest.thrift:190: seperated  ==> separated
> ./test/ThriftTest.thrift:198: seperated  ==> separated
> ./test/ThriftTest.thrift:206: seperated  ==> separated
> ./test/ThriftTest.thrift:214: seperated  ==> separated
> ./test/test.py:45: seperated  ==> separated
> ./test/test.py:47: seperated  ==> separated
> ./test/rb/core/transport/test_transport.rb:54: guarenteed  ==> guaranteed
> ./test/haxe/src/TestServerHandler.hx:121: seperated  ==> separated
> ./test/haxe/src/TestServerHandler.hx:161: seperated  ==> separated
> ./test/haxe/src/TestServerHandler.hx:186: seperated  ==> separated
> ./test/haxe/src/TestServerHandler.hx:211: seperated  ==> separated
> ./test/haxe/src/TestServerHandler.hx:236: seperated  ==> separated
> ./test/go/src/common/printing_handler.go:94: seperated  ==> separated
> ./test/go/src/common/printing_handler.go:118: seperated  ==> separated
> ./test/go/src/common/printing_handler.go:140: seperated  ==> separated
> ./test/go/src/common/printing_handler.go:162: seperated  ==> separated
> ./test/go/src/common/printing_handler.go:184: seperated  ==> separated
> WARNING: Binary file: ./test/keys/server.p12 
> WARNING: Binary file: ./test/keys/client.p12 
> ./test/hs/TestClient.hs:118: nestIn  ==> nesting
> ./test/hs/TestClient.hs:122: nestIn  ==> nesting
> ./test/hs/TestClient.hs:123: nestIn  ==> nesting
> ./aclocal/ax_javac_and_java.m4:10: currenly  ==> currently
> ./compiler/cpp/src/globals.h:126: seperately  ==> separately
> ./compiler/cpp/src/main.cc:995: Propogate  ==> Propagate
> ./compiler/cpp/src/parse/t_program.h:138: occurances  ==> occurrences
> ./compiler/cpp/src/parse/t_program.h:140: occurances  ==> occurrences
> ./compiler/cpp/src/parse/t_program.h:142: occurances  ==> occurrences
> ./compiler/cpp/src/parse/t_program.h:152: occurances  ==> occurrences
> ./compiler/cpp/src/parse/t_program.h:153: occurances  ==> occurrences
> ./compiler/cpp/src/parse/t_program.h:154: occurances  ==> occurrences
> ./compiler/cpp/src/parse/t_program.h:155: occurances  ==> occurrences
> ./compiler/cpp/src/parse/t_program.h:156: occurances  ==> occurrences
> ./compiler/cpp/src/parse/t_program.h:157: occurances  ==> occurrences
> ./compiler/cpp/src/parse/t_program.h:169: occurances  ==> occurrences
> ./compiler/cpp/src/parse/t_program.h:172: occurances  ==> occurrences
> ./compiler/cpp/src/parse/t_program.h:173: occurances  ==> occurrences
> ./compiler/cpp/src/generate/t_php_generator.cc:231: allready  ==> already
> ./compiler/cpp/src/generate/t_csharp_generator.cc:694: seperate  ==> separate
> ./compiler/cpp/src/generate/t_html_generator.cc:719: indentifier  ==> identifier
> ./lib/py/setup.py:105: occured  ==> occurred
> ./lib/py/src/protocol/fastbinary.c:335: FUCNTION  ==> FUNCTION
> ./lib/py/src/protocol/fastbinary.c:939: FUCNTION  ==> FUNCTION
> ./lib/py/src/server/TNonblockingServer.py:264: usualy  ==> usually
> ./lib/d/test/async_test.d:343: succeded  ==> succeeded
> ./lib/d/test/async_test.d:345: succeded  ==> succeeded
> ./lib/d/test/async_test.d:356: succeded  ==> succeeded
> ./lib/d/test/async_test.d:358: succeded  ==> succeeded
> ./lib/d/src/thrift/base.d:46: occured  ==> occurred
> ./lib/d/src/thrift/base.d:65: occured  ==> occurred
> ./lib/d/src/thrift/util/future.d:319: similiar  ==> similar
> ./lib/d/src/thrift/util/future.d:457: possiblity  ==> possibility
> ./lib/d/src/thrift/util/hashset.d:34: compatibilty  ==> compatibility
> ./lib/d/src/thrift/util/hashset.d:91: availabe  ==> available
> ./lib/d/src/thrift/async/libevent.d:433: excecuted  ==> executed
> ./lib/d/src/thrift/transport/file.d:249: accomodate  ==> accommodate
> ./lib/d/src/thrift/codegen/client.d:195: recieving  ==> receiving
> ./lib/d/src/thrift/codegen/client_pool.d:208: succeded  ==> succeeded
> ./lib/d/src/thrift/codegen/processor.d:257: recieving  ==> receiving
> ./lib/d/src/thrift/codegen/idlgen.d:158: occurences  ==> occurrences
> ./lib/d/src/thrift/codegen/idlgen.d:161: unneccesary  ==> unnecessary
> ./lib/d/src/thrift/server/base.d:40: occured  ==> occurred
> ./lib/rb/benchmark/thin_server.rb:31: upto  ==> up to
> ./lib/rb/benchmark/server.rb:32: upto  ==> up to
> ./lib/php/lib/Thrift/TMultiplexedProcessor.php:64: ususally  ==> usually
> ./lib/haxe/src/org/apache/thrift/protocol/TMultiplexedProcessor.hx:61: ususally  ==> usually
> WARNING: Binary file: ./lib/erl/rebar 
> ./lib/erl/src/thrift_json_parser.erl:222: preceeding  ==> preceding
> ./lib/go/thrift/socket.go:103: Retreive  ==> Retrieve
> ./lib/go/thrift/compact_protocol.go:301: preceeding  ==> preceding
> ./lib/go/thrift/compact_protocol.go:681: possiblity  ==> possibility
> ./lib/go/thrift/compact_protocol.go:687: possiblity  ==> possibility
> ./lib/go/thrift/ssl_socket.go:105: Retreive  ==> Retrieve
> ./lib/lua/Thrift.lua:72: achive  ==> achieve
> ./lib/ocaml/OCamlMakefile:262: cmo  ==> com  | disabled due to lots of false positives
> ./lib/ocaml/OCamlMakefile:457: cmo  ==> com  | disabled due to lots of false positives
> ./lib/ocaml/OCamlMakefile:458: cmo  ==> com  | disabled due to lots of false positives
> ./lib/ocaml/OCamlMakefile:461: cmo  ==> com  | disabled due to lots of false positives
> ./lib/ocaml/OCamlMakefile:464: cmo  ==> com  | disabled due to lots of false positives
> ./lib/ocaml/OCamlMakefile:465: cmo  ==> com  | disabled due to lots of false positives
> ./lib/ocaml/OCamlMakefile:466: cmo  ==> com  | disabled due to lots of false positives
> ./lib/ocaml/OCamlMakefile:467: cmo  ==> com  | disabled due to lots of false positives
> ./lib/ocaml/OCamlMakefile:470: cmo  ==> com  | disabled due to lots of false positives
> ./lib/ocaml/OCamlMakefile:491: cmo  ==> com  | disabled due to lots of false positives
> ./lib/ocaml/OCamlMakefile:493: cmo  ==> com  | disabled due to lots of false positives
> ./lib/ocaml/OCamlMakefile:494: cmo  ==> com  | disabled due to lots of false positives
> ./lib/ocaml/OCamlMakefile:607: cmo  ==> com  | disabled due to lots of false positives
> ./lib/ocaml/OCamlMakefile:870: cmo  ==> com  | disabled due to lots of false positives
> ./lib/ocaml/OCamlMakefile:922: cmo  ==> com  | disabled due to lots of false positives
> ./lib/ocaml/OCamlMakefile:958: cmo  ==> com  | disabled due to lots of false positives
> ./lib/ocaml/OCamlMakefile:959: cmo  ==> com  | disabled due to lots of false positives
> ./lib/ocaml/OCamlMakefile:965: cmo  ==> com  | disabled due to lots of false positives
> ./lib/ocaml/OCamlMakefile:966: cmo  ==> com  | disabled due to lots of false positives
> ./lib/ocaml/OCamlMakefile:1001: cmo  ==> com  | disabled due to lots of false positives
> ./lib/ocaml/OCamlMakefile:1016: cmo  ==> com  | disabled due to lots of false positives
> ./lib/ocaml/README-OCamlMakefile:203: maintainance  ==> maintenance
> ./lib/ocaml/README-OCamlMakefile:434: cmo  ==> com  | disabled due to lots of false positives
> ./lib/ocaml/README-OCamlMakefile:579: cmo  ==> com  | disabled due to lots of false positives
> ./lib/cpp/libthrift.vcxproj.filters:28: protocal  ==> protocol
> ./lib/cpp/libthrift.vcxproj.filters:31: protocal  ==> protocol
> ./lib/cpp/libthrift.vcxproj.filters:34: protocal  ==> protocol
> ./lib/cpp/libthrift.vcxproj.filters:37: protocal  ==> protocol
> ./lib/cpp/libthrift.vcxproj.filters:40: protocal  ==> protocol
> ./lib/cpp/libthrift.vcxproj.filters:123: protocal  ==> protocol
> ./lib/cpp/libthrift.vcxproj.filters:156: protocal  ==> protocol
> ./lib/cpp/libthrift.vcxproj.filters:159: protocal  ==> protocol
> ./lib/cpp/libthrift.vcxproj.filters:204: protocal  ==> protocol
> ./lib/cpp/libthrift.vcxproj.filters:207: protocal  ==> protocol
> ./lib/cpp/libthrift.vcxproj.filters:210: protocal  ==> protocol
> ./lib/cpp/libthrift.vcxproj.filters:213: protocal  ==> protocol
> ./lib/cpp/libthrift.vcxproj.filters:250: protocal  ==> protocol
> ./lib/cpp/libthrift.vcxproj.filters:277: protocal  ==> protocol
> ./lib/cpp/test/TransportTest.cpp:758: acheived  ==> achieved
> ./lib/cpp/src/thrift/TLogging.h:65: analagous  ==> analogous
> ./lib/cpp/src/thrift/TLogging.h:91: analagous  ==> analogous
> ./lib/cpp/src/thrift/TLogging.h:123: Analagous  ==> Analogous
> ./lib/cpp/src/thrift/concurrency/PosixThreadFactory.h:49: withing  ==> within
> ./lib/cpp/src/thrift/protocol/TCompactProtocol.tcc:264: preceeding  ==> preceding
> ./lib/cpp/src/thrift/protocol/TJSONProtocol.cpp:659: becuase  ==> because
> ./lib/cpp/src/thrift/protocol/TJSONProtocol.cpp:987: becuase  ==> because
> ./lib/cpp/src/thrift/protocol/TJSONProtocol.h:46: postive  ==> positive
> ./lib/cpp/src/thrift/transport/TBufferTransports.cpp:66: accomodate  ==> accommodate
> ./lib/cpp/src/thrift/transport/TSSLSocket.cpp:207: perfomance  ==> performance
> ./lib/cpp/src/thrift/transport/TSocket.h:189: formated  ==> formatted
> ./lib/cpp/src/thrift/transport/TBufferTransports.h:89: accomodate  ==> accommodate
> ./lib/cpp/src/thrift/transport/TBufferTransports.h:632: accomodate  ==> accommodate
> ./lib/cpp/src/thrift/server/TNonblockingServer.h:672: persistant  ==> persistent
> ./lib/cpp/src/thrift/server/TNonblockingServer.h:815: occured  ==> occurred
> ./lib/cpp/src/thrift/processor/TMultiplexedProcessor.h:99: ususally  ==> usually
> ./lib/nodejs/lib/thrift/web_server.js:296: embeded  ==> embedded
> ./lib/nodejs/lib/thrift/web_server.js:305: embeded  ==> embedded
> ./lib/nodejs/lib/thrift/web_server.js:508: availible  ==> available
> ./lib/nodejs/lib/thrift/server.js:31: Multipled  ==> Multiplied
> ./lib/nodejs/lib/thrift/server.js:100: Multipled  ==> Multiplied
> ./lib/java/test/log4j.properties:3: threshhold  ==> threshold
> WARNING: Binary file: ./lib/java/test/.keystore 
> WARNING: Binary file: ./lib/java/test/.truststore 
> ./lib/java/src/org/apache/thrift/TMultiplexedProcessor.java:63: ususally  ==> usually
> ./lib/cocoa/src/transport/TSSLSocketClient.m:196: occured  ==> occurred
> ./lib/cocoa/src/transport/TSSLSocketClient.m:197: occured  ==> occurred
> ./lib/ts/thrift.d.ts:222: suceeds  ==> succeeds
> ./lib/csharp/src/Protocol/TMultiplexedProcessor.cs:64: ususally  ==> usually
> ./lib/js/test/test.js:30: sucess  ==> success
> ./lib/js/src/thrift.js:1223: teh  ==> the
> ./lib/perl/README.md:31: upto  ==> up to
> ./lib/delphi/test/TestClient.pas:514: lenght  ==> length
> ./lib/delphi/test/skip/skiptest_version1.dpr:98: supressed  ==> suppressed
> ./lib/delphi/test/skip/skiptest_version1.dpr:125: supressed  ==> suppressed
> ./lib/delphi/test/skip/skiptest_version1.dpr:152: supressed  ==> suppressed
> ./lib/delphi/test/skip/README.md:6: compatibilty  ==> compatibility
> ./lib/delphi/test/skip/skiptest_version2.dpr:124: supressed  ==> suppressed
> ./lib/delphi/test/skip/skiptest_version2.dpr:152: supressed  ==> suppressed
> ./lib/delphi/test/skip/skiptest_version2.dpr:179: supressed  ==> suppressed
> ./lib/hs/README.md:92: proceding  ==> proceeding, preceding
> ./tutorial/c_glib/c_glib_server.c:111: occured  ==> occurred
> ./tutorial/c_glib/c_glib_client.c:80: occured  ==> occurred



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)