Skip to content

STEP8とSTEP13のリファクタリング #13

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 2 commits into from
Jun 13, 2023
Merged

Conversation

ShotaAk
Copy link

@ShotaAk ShotaAk commented Jun 13, 2023

What does this implement/fix?

straight関数の引数名を修正します。

SPIFFS.inoの未使用関数listDirを削除します。

Does this close any currently open issues?

いいえ

How has this been tested?

CIでビルドが通ることを確認します。

Any other comments?

Checklists

@ShotaAk ShotaAk added the Type: Refactoring A code change that neither fixes a bug nor adds a feature label Jun 13, 2023
@ShotaAk ShotaAk requested a review from aoki-rt June 13, 2023 05:51
@github-actions
Copy link

Memory usage change @ cc3dec4

Board flash % RAM for global variables %
esp32:esp32:esp32s3 💚 -40 - 0 -0.0 - 0.0 0 - 0 0.0 - 0.0
Click for full report table
Board uROS_STEP1_LED
flash
% uROS_STEP1_LED
RAM for global variables
% uROS_STEP2_SWITCH
flash
% uROS_STEP2_SWITCH
RAM for global variables
% uROS_STEP3_Buzzer
flash
% uROS_STEP3_Buzzer
RAM for global variables
% uROS_STEP4_Sensor
flash
% uROS_STEP4_Sensor
RAM for global variables
% uROS_STEP5_Straight
flash
% uROS_STEP5_Straight
RAM for global variables
% uROS_STEP6_rotate
flash
% uROS_STEP6_rotate
RAM for global variables
% uROS_STEP7_P_control
flash
% uROS_STEP7_P_control
RAM for global variables
% uROS_STEP8_micromouse
flash
% uROS_STEP8_micromouse
RAM for global variables
% uROS_STEP10_twistMsg
flash
% uROS_STEP10_twistMsg
RAM for global variables
% uROS_STEP11_tfMsg
flash
% uROS_STEP11_tfMsg
RAM for global variables
% uROS_STEP12_SensorMsg
flash
% uROS_STEP12_SensorMsg
RAM for global variables
% uROS_STEP13_micromouse
flash
% uROS_STEP13_micromouse
RAM for global variables
%
esp32:esp32:esp32s3 0 0.0 0 0.0 0 0.0 0 0.0 0 0.0 0 0.0 0 0.0 0 0.0 0 0.0 0 0.0 0 0.0 0 0.0 0 0.0 0 0.0 -36 -0.0 0 0.0 0 0.0 0 0.0 0 0.0 0 0.0 0 0.0 0 0.0 -40 -0.0 0 0.0
Click for full report CSV
Board,uROS_STEP1_LED<br>flash,%,uROS_STEP1_LED<br>RAM for global variables,%,uROS_STEP2_SWITCH<br>flash,%,uROS_STEP2_SWITCH<br>RAM for global variables,%,uROS_STEP3_Buzzer<br>flash,%,uROS_STEP3_Buzzer<br>RAM for global variables,%,uROS_STEP4_Sensor<br>flash,%,uROS_STEP4_Sensor<br>RAM for global variables,%,uROS_STEP5_Straight<br>flash,%,uROS_STEP5_Straight<br>RAM for global variables,%,uROS_STEP6_rotate<br>flash,%,uROS_STEP6_rotate<br>RAM for global variables,%,uROS_STEP7_P_control<br>flash,%,uROS_STEP7_P_control<br>RAM for global variables,%,uROS_STEP8_micromouse<br>flash,%,uROS_STEP8_micromouse<br>RAM for global variables,%,uROS_STEP10_twistMsg<br>flash,%,uROS_STEP10_twistMsg<br>RAM for global variables,%,uROS_STEP11_tfMsg<br>flash,%,uROS_STEP11_tfMsg<br>RAM for global variables,%,uROS_STEP12_SensorMsg<br>flash,%,uROS_STEP12_SensorMsg<br>RAM for global variables,%,uROS_STEP13_micromouse<br>flash,%,uROS_STEP13_micromouse<br>RAM for global variables,%
esp32:esp32:esp32s3,0,0.0,0,0.0,0,0.0,0,0.0,0,0.0,0,0.0,0,0.0,0,0.0,0,0.0,0,0.0,0,0.0,0,0.0,0,0.0,0,0.0,-36,-0.0,0,0.0,0,0.0,0,0.0,0,0.0,0,0.0,0,0.0,0,0.0,-40,-0.0,0,0.0

@aoki-rt
Copy link
Contributor

aoki-rt commented Jun 13, 2023

使用していないlistidir関数の削除、straigh関数の引数の変数名の変更後に重複しないことを確認しました。

@aoki-rt aoki-rt merged commit 52deef1 into main Jun 13, 2023
@ShotaAk ShotaAk deleted the refactor_STEP8_13 branch June 13, 2023 06:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Type: Refactoring A code change that neither fixes a bug nor adds a feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants