メイン電装フラッシュ統合 #33
メイン電装フラッシュ統合 #33
Conversation
![github-actions[bot]](https://avatars.githubusercontent.com/in/15368?s=60&v=4){kind=small}
There was a problem hiding this comment.
Remaining comments which cannot be posted as a review comment to avoid GitHub Rate Limit
cpplint
Main/ff.c|290| Missing space after , [whitespace/comma] [3]
Main/ff.c|291| Missing space after , [whitespace/comma] [3]
Main/ff.c|292| Missing space after , [whitespace/comma] [3]
Main/ff.c|293| Missing space after , [whitespace/comma] [3]
Main/ff.c|294| Missing space after , [whitespace/comma] [3]
Main/ff.c|295| Missing space after , [whitespace/comma] [3]
Main/ff.c|296| Missing space after , [whitespace/comma] [3]
Main/ff.c|297| Missing space after , [whitespace/comma] [3]
Main/ff.c|298| Missing space after , [whitespace/comma] [3]
Main/ff.c|299| Missing space after , [whitespace/comma] [3]
Main/ff.c|300| Missing space after , [whitespace/comma] [3]
Main/ff.c|301| Missing space after , [whitespace/comma] [3]
Main/ff.c|302| Missing space after , [whitespace/comma] [3]
Main/ff.c|303| Missing space after , [whitespace/comma] [3]
Main/ff.c|304| Missing space after , [whitespace/comma] [3]
Main/ff.c|305| Missing space after , [whitespace/comma] [3]
Main/ff.c|306| Missing space after , [whitespace/comma] [3]
Main/ff.c|307| Missing space after , [whitespace/comma] [3]
Main/ff.c|308| Missing space after , [whitespace/comma] [3]
Main/ff.c|309| Missing space after , [whitespace/comma] [3]
Main/ff.c|310| Missing space after , [whitespace/comma] [3]
Main/ff.c|311| Missing space after , [whitespace/comma] [3]
Main/ff.c|312| Missing space after , [whitespace/comma] [3]
Main/ff.c|313| Missing space after , [whitespace/comma] [3]
Main/ff.c|314| Missing space after , [whitespace/comma] [3]
Main/ff.c|315| Missing space after , [whitespace/comma] [3]
Main/ff.c|316| Missing space after , [whitespace/comma] [3]
Main/ff.c|317| Missing space after , [whitespace/comma] [3]
Main/ff.c|318| Missing space after , [whitespace/comma] [3]
Main/ff.c|319| Missing space after , [whitespace/comma] [3]
Main/ff.c|320| Missing space after , [whitespace/comma] [3]
Main/ff.c|321| Missing space after , [whitespace/comma] [3]
Main/ff.c|322| Missing space after , [whitespace/comma] [3]
Main/ff.c|323| Missing space after , [whitespace/comma] [3]
Main/ff.c|324| Missing space after , [whitespace/comma] [3]
Main/ff.c|325| Missing space after , [whitespace/comma] [3]
Main/ff.c|326| Missing space after , [whitespace/comma] [3]
Main/ff.c|327| Missing space after , [whitespace/comma] [3]
Main/ff.c|328| Missing space after , [whitespace/comma] [3]
Main/ff.c|329| Missing space after , [whitespace/comma] [3]
Main/ff.c|330| Missing space after , [whitespace/comma] [3]
Main/ff.c|331| Missing space after , [whitespace/comma] [3]
Main/ff.c|332| Missing space after , [whitespace/comma] [3]
Main/ff.c|333| Missing space after , [whitespace/comma] [3]
Main/ff.c|334| Missing space after , [whitespace/comma] [3]
Main/ff.c|335| Missing space after , [whitespace/comma] [3]
Main/ff.c|336| Missing space after , [whitespace/comma] [3]
Main/ff.c|337| Missing space after , [whitespace/comma] [3]
Main/ff.c|338| Missing space after , [whitespace/comma] [3]
Main/ff.c|339| Missing space after , [whitespace/comma] [3]
Main/ff.c|340| Missing space after , [whitespace/comma] [3]
Main/ff.c|341| Missing space after , [whitespace/comma] [3]
Main/ff.c|342| Missing space after , [whitespace/comma] [3]
Main/ff.c|343| Missing space after , [whitespace/comma] [3]
Main/ff.c|344| Missing space after , [whitespace/comma] [3]
Main/ff.c|345| Missing space after , [whitespace/comma] [3]
Main/ff.c|346| Missing space after , [whitespace/comma] [3]
Main/ff.c|347| Missing space after , [whitespace/comma] [3]
Main/ff.c|348| Missing space after , [whitespace/comma] [3]
Main/ff.c|349| Missing space after , [whitespace/comma] [3]
Main/ff.c|350| Missing space after , [whitespace/comma] [3]
Main/ff.c|351| Missing space after , [whitespace/comma] [3]
Main/ff.c|352| Missing space after , [whitespace/comma] [3]
Main/ff.c|353| Missing space after , [whitespace/comma] [3]
Main/ff.c|354| Missing space after , [whitespace/comma] [3]
Main/ff.c|355| Missing space after , [whitespace/comma] [3]
Main/ff.c|356| Missing space after , [whitespace/comma] [3]
Main/ff.c|357| Missing space after , [whitespace/comma] [3]
Main/ff.c|358| Missing space after , [whitespace/comma] [3]
Main/ff.c|359| Missing space after , [whitespace/comma] [3]
Main/ff.c|360| Missing space after , [whitespace/comma] [3]
Main/ff.c|361| Missing space after , [whitespace/comma] [3]
Main/ff.c|362| Missing space after , [whitespace/comma] [3]
Main/ff.c|363| Missing space after , [whitespace/comma] [3]
Main/ff.c|364| Missing space after , [whitespace/comma] [3]
Main/ff.c|365| Missing space after , [whitespace/comma] [3]
Main/ff.c|366| Missing space after , [whitespace/comma] [3]
Main/ff.c|367| Missing space after , [whitespace/comma] [3]
Main/ff.c|368| Missing space after , [whitespace/comma] [3]
Main/ff.c|369| Missing space after , [whitespace/comma] [3]
Main/ff.c|370| Missing space after , [whitespace/comma] [3]
Main/ff.c|371| Missing space after , [whitespace/comma] [3]
Main/ff.c|372| Missing space after , [whitespace/comma] [3]
Main/ff.c|373| Missing space after , [whitespace/comma] [3]
Main/ff.c|374| Missing space after , [whitespace/comma] [3]
Main/ff.c|375| Missing space after , [whitespace/comma] [3]
Main/ff.c|376| Missing space after , [whitespace/comma] [3]
Main/ff.c|377| Missing space after , [whitespace/comma] [3]
Main/ff.c|378| Missing space after , [whitespace/comma] [3]
Main/ff.c|379| Missing space after , [whitespace/comma] [3]
Main/ff.c|380| Missing space after , [whitespace/comma] [3]
Main/ff.c|381| Missing space after , [whitespace/comma] [3]
Main/ff.c|382| Missing space after , [whitespace/comma] [3]
Main/ff.c|383| Missing space after , [whitespace/comma] [3]
Main/ff.c|384| Missing space after , [whitespace/comma] [3]
Main/ff.c|385| Missing space after , [whitespace/comma] [3]
Main/ff.c|386| Missing space after , [whitespace/comma] [3]
Main/ff.c|387| Missing space after , [whitespace/comma] [3]
Main/ff.c|388| Missing space after , [whitespace/comma] [3]
Main/ff.c|389| Missing space after , [whitespace/comma] [3]
Main/ff.c|390| Missing space after , [whitespace/comma] [3]
Main/ff.c|391| Missing space after , [whitespace/comma] [3]
Main/ff.c|392| Missing space after , [whitespace/comma] [3]
Main/ff.c|393| Missing space after , [whitespace/comma] [3]
Main/ff.c|394| Missing space after , [whitespace/comma] [3]
Main/ff.c|395| Missing space after , [whitespace/comma] [3]
Main/ff.c|396| Missing space after , [whitespace/comma] [3]
Main/ff.c|397| Missing space after , [whitespace/comma] [3]
Main/ff.c|398| Missing space after , [whitespace/comma] [3]
Main/ff.c|399| Missing space after , [whitespace/comma] [3]
Main/ff.c|400| Missing space after , [whitespace/comma] [3]
Main/ff.c|401| Missing space after , [whitespace/comma] [3]
Main/ff.c|402| Missing space after , [whitespace/comma] [3]
Main/ff.c|403| Missing space after , [whitespace/comma] [3]
Main/ff.c|404| Missing space after , [whitespace/comma] [3]
Main/ff.c|405| Missing space after , [whitespace/comma] [3]
Main/ff.c|479| Missing space after , [whitespace/comma] [3]
Main/ff.c|507| Using C-style cast. Use reinterpret_cast<BYTE*>(...) instead [readability/casting] [4]
Main/ff.c|656| Using C-style cast. Use reinterpret_cast<BYTE*>(...) instead [readability/casting] [4]
Main/ff.c|670| Using C-style cast. Use reinterpret_cast<BYTE*>(...) instead [readability/casting] [4]
Main/ff.c|891| Using C-style cast. Use static_cast(...) instead [readability/casting] [4]
Main/ff.c|1351| Using C-style cast. Use static_cast(...) instead [readability/casting] [4]
Main/ff.c|1809| Using C-style cast. Use static_cast(...) instead [readability/casting] [4]
Main/ff.c|2372| Using C-style cast. Use static_cast(...) instead [readability/casting] [4]
Main/ff.c|2391| Using C-style cast. Use static_cast(...) instead [readability/casting] [4]
Main/ff.c|3130| Using C-style cast. Use static_cast(...) instead [readability/casting] [4]
Main/ff.c|3747| Using C-style cast. Use reinterpret_cast<BYTE*>(...) instead [readability/casting] [4]
Main/ff.c|5280| Using C-style cast. Use static_cast(...) instead [readability/casting] [4]
Main/ff.c|5304| Using C-style cast. Use static_cast(...) instead [readability/casting] [4]
Main/ff.c|5460| Missing space after , [whitespace/comma] [3]
Main/ff.c|5583| Using C-style cast. Use reinterpret_cast<BYTE*>(...) instead [readability/casting] [4]
Main/ff.c|5784| Redundant blank line at the end of a code block should be deleted. [whitespace/blank_line] [3]
Main/ff.c|6001| Using C-style cast. Use reinterpret_cast<BYTE*>(...) instead [readability/casting] [4]
Main/ff.c|6012| Using C-style cast. Use reinterpret_cast<BYTE*>(...) instead [readability/casting] [4]
Main/ff.c|6271| Using C-style cast. Use reinterpret_cast<TCHAR*>(...) instead [readability/casting] [4]
Main/ff.c|6340| Using C-style cast. Use static_cast(...) instead [readability/casting] [4]
Main/ff.c|6467| Extra space before last semicolon. If this should be an empty statement, use {} instead. [whitespace/semicolon] [5]
Main/ff.c|6469| Extra space before last semicolon. If this should be an empty statement, use {} instead. [whitespace/semicolon] [5]
Main/ff.c|6470| Extra space before last semicolon. If this should be an empty statement, use {} instead. [whitespace/semicolon] [5]
Main/ff.c|6494| Use int16/int64/etc, rather than the C type long [runtime/int] [4]
Main/ff.h|29| Include the directory when naming header files [build/include_subdir] [4]
Main/ff.h|55| Use int16/int64/etc, rather than the C type short [runtime/int] [4]
Main/ff.h|56| Use int16/int64/etc, rather than the C type short [runtime/int] [4]
Main/ff.h|57| Use int16/int64/etc, rather than the C type long [runtime/int] [4]
Main/ff.h|307| Missing space after , [whitespace/comma] [3]
Main/ff.h|318| Using C-style cast. Use static_cast(...) instead [readability/casting] [4]
Main/ffconf.h|171| Missing space after , [whitespace/comma] [3]
Main/ffconf.h|265| Complex multi-line /.../-style comment found. Lint may give bogus warnings. Consider replacing these with //-style comments, with #if 0...#endif, or with more clearly structured multi-line comments. [readability/multiline_comment] [5]
test/Main_SdFat_format/Main_SdFat_format.ino|27| Include the directory when naming header files [build/include_subdir] [4]
test/Main_SdFat_format/Main_SdFat_format.ino|28| Include the directory when naming header files [build/include_subdir] [4]
test/Main_SdFat_format/Main_SdFat_format.ino|129| Using C-style cast. Use const_cast<char >(...) instead [readability/casting] [4]
test/Main_SdFat_format/Main_SdFat_format.ino|195| Using C-style cast. Use reinterpret_cast<DWORD >(...) instead [readability/casting] [4]
test/Main_SdFat_format/Main_SdFat_format.ino|199| Using C-style cast. Use reinterpret_cast<WORD >(...) instead [readability/casting] [4]
test/Main_SdFat_format/Main_SdFat_format.ino|203| Using C-style cast. Use reinterpret_cast<DWORD >(...) instead [readability/casting] [4]
test/Main_SdFat_format/ff.c|22| Include the directory when naming header files [build/include_subdir] [4]
test/Main_SdFat_format/ff.c|23| Include the directory when naming header files [build/include_subdir] [4]
test/Main_SdFat_format/ff.c|270| Missing space after , [whitespace/comma] [3]
test/Main_SdFat_format/ff.c|271| Missing space after , [whitespace/comma] [3]
test/Main_SdFat_format/ff.c|272| Missing space after , [whitespace/comma] [3]
test/Main_SdFat_format/ff.c|273| Missing space after , [whitespace/comma] [3]
test/Main_SdFat_format/ff.c|274| Missing space after , [whitespace/comma] [3]
test/Main_SdFat_format/ff.c|275| Missing space after , [whitespace/comma] [3]
test/Main_SdFat_format/ff.c|276| Missing space after , [whitespace/comma] [3]
test/Main_SdFat_format/ff.c|277| Missing space after , [whitespace/comma] [3]
test/Main_SdFat_format/ff.c|278| Missing space after , [whitespace/comma] [3]
test/Main_SdFat_format/ff.c|279| Missing space after , [whitespace/comma] [3]
test/Main_SdFat_format/ff.c|280| Missing space after , [whitespace/comma] [3]
test/Main_SdFat_format/ff.c|281| Missing space after , [whitespace/comma] [3]
test/Main_SdFat_format/ff.c|282| Missing space after , [whitespace/comma] [3]
test/Main_SdFat_format/ff.c|283| Missing space after , [whitespace/comma] [3]
test/Main_SdFat_format/ff.c|284| Missing space after , [whitespace/comma] [3]
test/Main_SdFat_format/ff.c|285| Missing space after , [whitespace/comma] [3]
test/Main_SdFat_format/ff.c|286| Missing space after , [whitespace/comma] [3]
test/Main_SdFat_format/ff.c|287| Missing space after , [whitespace/comma] [3]
test/Main_SdFat_format/ff.c|288| Missing space after , [whitespace/comma] [3]
test/Main_SdFat_format/ff.c|289| Missing space after , [whitespace/comma] [3]
test/Main_SdFat_format/ff.c|290| Missing space after , [whitespace/comma] [3]
test/Main_SdFat_format/ff.c|291| Missing space after , [whitespace/comma] [3]
test/Main_SdFat_format/ff.c|292| Missing space after , [whitespace/comma] [3]
test/Main_SdFat_format/ff.c|293| Missing space after , [whitespace/comma] [3]
test/Main_SdFat_format/ff.c|294| Missing space after , [whitespace/comma] [3]
test/Main_SdFat_format/ff.c|295| Missing space after , [whitespace/comma] [3]
test/Main_SdFat_format/ff.c|296| Missing space after , [whitespace/comma] [3]
test/Main_SdFat_format/ff.c|297| Missing space after , [whitespace/comma] [3]
test/Main_SdFat_format/ff.c|298| Missing space after , [whitespace/comma] [3]
test/Main_SdFat_format/ff.c|299| Missing space after , [whitespace/comma] [3]
test/Main_SdFat_format/ff.c|300| Missing space after , [whitespace/comma] [3]
test/Main_SdFat_format/ff.c|301| Missing space after , [whitespace/comma] [3]
test/Main_SdFat_format/ff.c|302| Missing space after , [whitespace/comma] [3]
test/Main_SdFat_format/ff.c|303| Missing space after , [whitespace/comma] [3]
test/Main_SdFat_format/ff.c|304| Missing space after , [whitespace/comma] [3]
test/Main_SdFat_format/ff.c|305| Missing space after , [whitespace/comma] [3]
test/Main_SdFat_format/ff.c|306| Missing space after , [whitespace/comma] [3]
test/Main_SdFat_format/ff.c|307| Missing space after , [whitespace/comma] [3]
test/Main_SdFat_format/ff.c|308| Missing space after , [whitespace/comma] [3]
test/Main_SdFat_format/ff.c|309| Missing space after , [whitespace/comma] [3]
test/Main_SdFat_format/ff.c|310| Missing space after , [whitespace/comma] [3]
test/Main_SdFat_format/ff.c|311| Missing space after , [whitespace/comma] [3]
test/Main_SdFat_format/ff.c|312| Missing space after , [whitespace/comma] [3]
test/Main_SdFat_format/ff.c|313| Missing space after , [whitespace/comma] [3]
test/Main_SdFat_format/ff.c|314| Missing space after , [whitespace/comma] [3]
test/Main_SdFat_format/ff.c|315| Missing space after , [whitespace/comma] [3]
test/Main_SdFat_format/ff.c|316| Missing space after , [whitespace/comma] [3]
test/Main_SdFat_format/ff.c|317| Missing space after , [whitespace/comma] [3]
test/Main_SdFat_format/ff.c|318| Missing space after , [whitespace/comma] [3]
test/Main_SdFat_format/ff.c|319| Missing space after , [whitespace/comma] [3]
test/Main_SdFat_format/ff.c|320| Missing space after , [whitespace/comma] [3]
test/Main_SdFat_format/ff.c|321| Missing space after , [whitespace/comma] [3]
test/Main_SdFat_format/ff.c|322| Missing space after , [whitespace/comma] [3]
test/Main_SdFat_format/ff.c|323| Missing space after , [whitespace/comma] [3]
test/Main_SdFat_format/ff.c|324| Missing space after , [whitespace/comma] [3]
test/Main_SdFat_format/ff.c|325| Missing space after , [whitespace/comma] [3]
test/Main_SdFat_format/ff.c|326| Missing space after , [whitespace/comma] [3]
test/Main_SdFat_format/ff.c|327| Missing space after , [whitespace/comma] [3]
test/Main_SdFat_format/ff.c|328| Missing space after , [whitespace/comma] [3]
test/Main_SdFat_format/ff.c|329| Missing space after , [whitespace/comma] [3]
test/Main_SdFat_format/ff.c|330| Missing space after , [whitespace/comma] [3]
test/Main_SdFat_format/ff.c|331| Missing space after , [whitespace/comma] [3]
test/Main_SdFat_format/ff.c|332| Missing space after , [whitespace/comma] [3]
test/Main_SdFat_format/ff.c|333| Missing space after , [whitespace/comma] [3]
test/Main_SdFat_format/ff.c|334| Missing space after , [whitespace/comma] [3]
test/Main_SdFat_format/ff.c|335| Missing space after , [whitespace/comma] [3]
test/Main_SdFat_format/ff.c|336| Missing space after , [whitespace/comma] [3]
test/Main_SdFat_format/ff.c|337| Missing space after , [whitespace/comma] [3]
test/Main_SdFat_format/ff.c|338| Missing space after , [whitespace/comma] [3]
test/Main_SdFat_format/ff.c|339| Missing space after , [whitespace/comma] [3]
test/Main_SdFat_format/ff.c|340| Missing space after , [whitespace/comma] [3]
test/Main_SdFat_format/ff.c|341| Missing space after , [whitespace/comma] [3]
test/Main_SdFat_format/ff.c|342| Missing space after , [whitespace/comma] [3]
test/Main_SdFat_format/ff.c|343| Missing space after , [whitespace/comma] [3]
test/Main_SdFat_format/ff.c|344| Missing space after , [whitespace/comma] [3]
test/Main_SdFat_format/ff.c|345| Missing space after , [whitespace/comma] [3]
test/Main_SdFat_format/ff.c|346| Missing space after , [whitespace/comma] [3]
test/Main_SdFat_format/ff.c|347| Missing space after , [whitespace/comma] [3]
test/Main_SdFat_format/ff.c|348| Missing space after , [whitespace/comma] [3]
test/Main_SdFat_format/ff.c|349| Missing space after , [whitespace/comma] [3]
test/Main_SdFat_format/ff.c|350| Missing space after , [whitespace/comma] [3]
test/Main_SdFat_format/ff.c|351| Missing space after , [whitespace/comma] [3]
test/Main_SdFat_format/ff.c|352| Missing space after , [whitespace/comma] [3]
test/Main_SdFat_format/ff.c|353| Missing space after , [whitespace/comma] [3]
test/Main_SdFat_format/ff.c|354| Missing space after , [whitespace/comma] [3]
test/Main_SdFat_format/ff.c|355| Missing space after , [whitespace/comma] [3]
test/Main_SdFat_format/ff.c|356| Missing space after , [whitespace/comma] [3]
test/Main_SdFat_format/ff.c|357| Missing space after , [whitespace/comma] [3]
test/Main_SdFat_format/ff.c|358| Missing space after , [whitespace/comma] [3]
test/Main_SdFat_format/ff.c|359| Missing space after , [whitespace/comma] [3]
test/Main_SdFat_format/ff.c|360| Missing space after , [whitespace/comma] [3]
test/Main_SdFat_format/ff.c|361| Missing space after , [whitespace/comma] [3]
test/Main_SdFat_format/ff.c|362| Missing space after , [whitespace/comma] [3]
test/Main_SdFat_format/ff.c|363| Missing space after , [whitespace/comma] [3]
test/Main_SdFat_format/ff.c|364| Missing space after , [whitespace/comma] [3]
test/Main_SdFat_format/ff.c|365| Missing space after , [whitespace/comma] [3]
test/Main_SdFat_format/ff.c|366| Missing space after , [whitespace/comma] [3]
test/Main_SdFat_format/ff.c|367| Missing space after , [whitespace/comma] [3]
test/Main_SdFat_format/ff.c|368| Missing space after , [whitespace/comma] [3]
test/Main_SdFat_format/ff.c|369| Missing space after , [whitespace/comma] [3]
test/Main_SdFat_format/ff.c|370| Missing space after , [whitespace/comma] [3]
test/Main_SdFat_format/ff.c|371| Missing space after , [whitespace/comma] [3]
test/Main_SdFat_format/ff.c|372| Missing space after , [whitespace/comma] [3]
test/Main_SdFat_format/ff.c|373| Missing space after , [whitespace/comma] [3]
test/Main_SdFat_format/ff.c|374| Missing space after , [whitespace/comma] [3]
test/Main_SdFat_format/ff.c|375| Missing space after , [whitespace/comma] [3]
test/Main_SdFat_format/ff.c|376| Missing space after , [whitespace/comma] [3]
test/Main_SdFat_format/ff.c|377| Missing space after , [whitespace/comma] [3]
test/Main_SdFat_format/ff.c|378| Missing space after , [whitespace/comma] [3]
test/Main_SdFat_format/ff.c|379| Missing space after , [whitespace/comma] [3]
test/Main_SdFat_format/ff.c|380| Missing space after , [whitespace/comma] [3]
test/Main_SdFat_format/ff.c|381| Missing space after , [whitespace/comma] [3]
test/Main_SdFat_format/ff.c|382| Missing space after , [whitespace/comma] [3]
test/Main_SdFat_format/ff.c|383| Missing space after , [whitespace/comma] [3]
test/Main_SdFat_format/ff.c|384| Missing space after , [whitespace/comma] [3]
test/Main_SdFat_format/ff.c|385| Missing space after , [whitespace/comma] [3]
test/Main_SdFat_format/ff.c|386| Missing space after , [whitespace/comma] [3]
test/Main_SdFat_format/ff.c|387| Missing space after , [whitespace/comma] [3]
test/Main_SdFat_format/ff.c|388| Missing space after , [whitespace/comma] [3]
test/Main_SdFat_format/ff.c|389| Missing space after , [whitespace/comma] [3]
test/Main_SdFat_format/ff.c|390| Missing space after , [whitespace/comma] [3]
test/Main_SdFat_format/ff.c|391| Missing space after , [whitespace/comma] [3]
test/Main_SdFat_format/ff.c|392| Missing space after , [whitespace/comma] [3]
test/Main_SdFat_format/ff.c|393| Missing space after , [whitespace/comma] [3]
test/Main_SdFat_format/ff.c|394| Missing space after , [whitespace/comma] [3]
test/Main_SdFat_format/ff.c|395| Missing space after , [whitespace/comma] [3]
test/Main_SdFat_format/ff.c|396| Missing space after , [whitespace/comma] [3]
test/Main_SdFat_format/ff.c|397| Missing space after , [whitespace/comma] [3]
test/Main_SdFat_format/ff.c|398| Missing space after , [whitespace/comma] [3]
test/Main_SdFat_format/ff.c|399| Missing space after , [whitespace/comma] [3]
test/Main_SdFat_format/ff.c|400| Missing space after , [whitespace/comma] [3]
test/Main_SdFat_format/ff.c|401| Missing space after , [whitespace/comma] [3]
test/Main_SdFat_format/ff.c|402| Missing space after , [whitespace/comma] [3]
test/Main_SdFat_format/ff.c|403| Missing space after , [whitespace/comma] [3]
test/Main_SdFat_format/ff.c|404| Missing space after , [whitespace/comma] [3]
test/Main_SdFat_format/ff.c|405| Missing space after , [whitespace/comma] [3]
test/Main_SdFat_format/ff.c|479| Missing space after , [whitespace/comma] [3]
test/Main_SdFat_format/ff.c|507| Using C-style cast. Use reinterpret_cast<BYTE>(...) instead [readability/casting] [4]
test/Main_SdFat_format/ff.c|656| Using C-style cast. Use reinterpret_cast<BYTE>(...) instead [readability/casting] [4]
test/Main_SdFat_format/ff.c|670| Using C-style cast. Use reinterpret_cast<BYTE>(...) instead [readability/casting] [4]
test/Main_SdFat_format/ff.c|891| Using C-style cast. Use static_cast(...) instead [readability/casting] [4]
test/Main_SdFat_format/ff.c|1351| Using C-style cast. Use static_cast(...) instead [readability/casting] [4]
test/Main_SdFat_format/ff.c|1809| Using C-style cast. Use static_cast(...) instead [readability/casting] [4]
test/Main_SdFat_format/ff.c|2372| Using C-style cast. Use static_cast(...) instead [readability/casting] [4]
test/Main_SdFat_format/ff.c|2391| Using C-style cast. Use static_cast(...) instead [readability/casting] [4]
test/Main_SdFat_format/ff.c|3130| Using C-style cast. Use static_cast(...) instead [readability/casting] [4]
test/Main_SdFat_format/ff.c|3747| Using C-style cast. Use reinterpret_cast<BYTE>(...) instead [readability/casting] [4]
test/Main_SdFat_format/ff.c|5280| Using C-style cast. Use static_cast(...) instead [readability/casting] [4]
test/Main_SdFat_format/ff.c|5304| Using C-style cast. Use static_cast(...) instead [readability/casting] [4]
test/Main_SdFat_format/ff.c|5460| Missing space after , [whitespace/comma] [3]
test/Main_SdFat_format/ff.c|5583| Using C-style cast. Use reinterpret_cast<BYTE*>(...) instead [readability/casting] [4]
test/Main_SdFat_format/ff.c|5784| Redundant blank line at the end of a code block should be deleted. [whitespace/blank_line] [3]
test/Main_SdFat_format/ff.c|6001| Using C-style cast. Use reinterpret_cast<BYTE*>(...) instead [readability/casting] [4]
test/Main_SdFat_format/ff.c|6012| Using C-style cast. Use reinterpret_cast<BYTE*>(...) instead [readability/casting] [4]
test/Main_SdFat_format/ff.c|6271| Using C-style cast. Use reinterpret_cast<TCHAR*>(...) instead [readability/casting] [4]
test/Main_SdFat_format/ff.c|6340| Using C-style cast. Use static_cast(...) instead [readability/casting] [4]
test/Main_SdFat_format/ff.c|6467| Extra space before last semicolon. If this should be an empty statement, use {} instead. [whitespace/semicolon] [5]
test/Main_SdFat_format/ff.c|6469| Extra space before last semicolon. If this should be an empty statement, use {} instead. [whitespace/semicolon] [5]
test/Main_SdFat_format/ff.c|6470| Extra space before last semicolon. If this should be an empty statement, use {} instead. [whitespace/semicolon] [5]
test/Main_SdFat_format/ff.c|6494| Use int16/int64/etc, rather than the C type long [runtime/int] [4]
test/Main_SdFat_format/ff.h|29| Include the directory when naming header files [build/include_subdir] [4]
test/Main_SdFat_format/ff.h|55| Use int16/int64/etc, rather than the C type short [runtime/int] [4]
test/Main_SdFat_format/ff.h|56| Use int16/int64/etc, rather than the C type short [runtime/int] [4]
test/Main_SdFat_format/ff.h|57| Use int16/int64/etc, rather than the C type long [runtime/int] [4]
test/Main_SdFat_format/ff.h|307| Missing space after , [whitespace/comma] [3]
test/Main_SdFat_format/ff.h|318| Using C-style cast. Use static_cast(...) instead [readability/casting] [4]
test/Main_SdFat_format/ffconf.h|171| Missing space after , [whitespace/comma] [3]
test/Main_SdFat_format/ffconf.h|265| Complex multi-line /.../-style comment found. Lint may give bogus warnings. Consider replacing these with //-style comments, with #if 0...#endif, or with more clearly structured multi-line comments. [readability/multiline_comment] [5]
| static semaphore_t sem_core1_using_buf_B; | ||
| static semaphore_t sem_need_format; | ||
| #define FILE_NAME "log.csv" | ||
| #include "ff.h" |
There was a problem hiding this comment.
[cpplint] reported by reviewdog 🐶
Include the directory when naming header files [build/include_subdir] [4]
| static semaphore_t sem_need_format; | ||
| #define FILE_NAME "log.csv" | ||
| #include "ff.h" | ||
| #include "diskio.h" |
There was a problem hiding this comment.
[cpplint] reported by reviewdog 🐶
Include the directory when naming header files [build/include_subdir] [4]
| //Serial.println("lock"); | ||
| return; | ||
| } | ||
| memcpy((void *)&SD_buf[SD_buf_index][SD_buf_count[SD_buf_index]], (void *)str, str_len * sizeof(char)); |
There was a problem hiding this comment.
[cpplint] reported by reviewdog 🐶
Using C-style cast. Use reinterpret_cast<void *>(...) instead [readability/casting] [4]
| dataFile = fatfs.open(FILE_NAME, FILE_WRITE); | ||
| } | ||
| if (dataFile) { | ||
| dataFile.write((char *)SD_buf[0], sizeof(char) * (SD_buf_count[0])); |
There was a problem hiding this comment.
[cpplint] reported by reviewdog 🐶
Using C-style cast. Use reinterpret_cast<char *>(...) instead [readability/casting] [4]
| dataFile = fatfs.open(FILE_NAME, FILE_WRITE); | ||
| } | ||
| if (dataFile) { | ||
| dataFile.write((char *)SD_buf[1], sizeof(char) * (SD_buf_count[1])); |
There was a problem hiding this comment.
[cpplint] reported by reviewdog 🐶
Using C-style cast. Use reinterpret_cast<char *>(...) instead [readability/casting] [4]
| 0xC0,0xC1,0xC2,0xC3,0xC4,0xC5,0xC6,0xC7,0xC8,0xC9,0xCA,0xCB,0xCC,0xCD,0xCE,0xCF, \ | ||
| 0xD0,0xD1,0xD2,0xD3,0xD4,0xD5,0xD6,0xD7,0xD8,0xD9,0xDA,0xDB,0xDC,0xDD,0xDE,0xDF, \ | ||
| 0xE0,0xE1,0xE2,0xE3,0xE4,0xE5,0xE6,0xE7,0xE8,0xE9,0xEA,0xEB,0xEC,0xED,0xEE,0xEF, \ | ||
| 0xF0,0xF1,0xF2,0xF3,0xF4,0xF5,0xF6,0xF7,0xF8,0xF9,0xFA,0xFB,0xFC,0xFD,0xFE,0xFF} |
There was a problem hiding this comment.
[cpplint] reported by reviewdog 🐶
Missing space after , [whitespace/comma] [3]
| 0xD0,0xD1,0xD2,0xD3,0xD4,0xD5,0xD6,0xD7,0xD8,0xD9,0xDA,0xDB,0xDC,0xDD,0xDE,0xDF, \ | ||
| 0xE0,0xE1,0xE2,0xE3,0xE4,0xE5,0xE6,0xE7,0xE8,0xE9,0xEA,0xEB,0xEC,0xED,0xEE,0xEF, \ | ||
| 0xF0,0xF1,0xF2,0xF3,0xF4,0xF5,0xF6,0xF7,0xF8,0xF9,0xFA,0xFB,0xFC,0xFD,0xFE,0xFF} | ||
| #define TBL_CT737 {0x80,0x81,0x82,0x83,0x84,0x85,0x86,0x87,0x88,0x89,0x8A,0x8B,0x8C,0x8D,0x8E,0x8F, \ |
There was a problem hiding this comment.
[cpplint] reported by reviewdog 🐶
Missing space after , [whitespace/comma] [3]
| 0xE0,0xE1,0xE2,0xE3,0xE4,0xE5,0xE6,0xE7,0xE8,0xE9,0xEA,0xEB,0xEC,0xED,0xEE,0xEF, \ | ||
| 0xF0,0xF1,0xF2,0xF3,0xF4,0xF5,0xF6,0xF7,0xF8,0xF9,0xFA,0xFB,0xFC,0xFD,0xFE,0xFF} | ||
| #define TBL_CT737 {0x80,0x81,0x82,0x83,0x84,0x85,0x86,0x87,0x88,0x89,0x8A,0x8B,0x8C,0x8D,0x8E,0x8F, \ | ||
| 0x90,0x92,0x92,0x93,0x94,0x95,0x96,0x97,0x80,0x81,0x82,0x83,0x84,0x85,0x86,0x87, \ |
There was a problem hiding this comment.
[cpplint] reported by reviewdog 🐶
Missing space after , [whitespace/comma] [3]
| 0xF0,0xF1,0xF2,0xF3,0xF4,0xF5,0xF6,0xF7,0xF8,0xF9,0xFA,0xFB,0xFC,0xFD,0xFE,0xFF} | ||
| #define TBL_CT737 {0x80,0x81,0x82,0x83,0x84,0x85,0x86,0x87,0x88,0x89,0x8A,0x8B,0x8C,0x8D,0x8E,0x8F, \ | ||
| 0x90,0x92,0x92,0x93,0x94,0x95,0x96,0x97,0x80,0x81,0x82,0x83,0x84,0x85,0x86,0x87, \ | ||
| 0x88,0x89,0x8A,0x8B,0x8C,0x8D,0x8E,0x8F,0x90,0x91,0xAA,0x92,0x93,0x94,0x95,0x96, \ |
There was a problem hiding this comment.
[cpplint] reported by reviewdog 🐶
Missing space after , [whitespace/comma] [3]
| #define TBL_CT737 {0x80,0x81,0x82,0x83,0x84,0x85,0x86,0x87,0x88,0x89,0x8A,0x8B,0x8C,0x8D,0x8E,0x8F, \ | ||
| 0x90,0x92,0x92,0x93,0x94,0x95,0x96,0x97,0x80,0x81,0x82,0x83,0x84,0x85,0x86,0x87, \ | ||
| 0x88,0x89,0x8A,0x8B,0x8C,0x8D,0x8E,0x8F,0x90,0x91,0xAA,0x92,0x93,0x94,0x95,0x96, \ | ||
| 0xB0,0xB1,0xB2,0xB3,0xB4,0xB5,0xB6,0xB7,0xB8,0xB9,0xBA,0xBB,0xBC,0xBD,0xBE,0xBF, \ |
There was a problem hiding this comment.
[cpplint] reported by reviewdog 🐶
Missing space after , [whitespace/comma] [3]
|
こんな感じで厳しい
UARTでMIFに送ってMIFに乗ってるCanvasからずっと使ってるフラッシュに文字列を1byteずつチマチマ書く方式でいく |
守屋のコードをもとに統合