Skip to content

Commit 7439e49

Browse files
committed
Fix Arduino example usage without progmem usage
1 parent 7370da2 commit 7439e49

2 files changed

Lines changed: 31 additions & 31 deletions

File tree

examples/arduino/basic_usage/src/main.cpp

Lines changed: 20 additions & 20 deletions
Original file line numberDiff line numberDiff line change
@@ -76,7 +76,7 @@ void setup()
7676

7777
// CLI init to use Serial as interface
7878
Cli.setup(&Serial, SERIAL_BAUDS);
79-
Cli.printf(PSTR("\nCommand Line Interface is ready\n\n"));
79+
Cli.printf("\nCommand Line Interface is ready\n\n");
8080
}
8181

8282
void loop()
@@ -87,21 +87,21 @@ void loop()
8787
if(Cli.manage(&cli_read))
8888
{
8989
// Show read result element
90-
Cli.printf(PSTR("Command received: %s\n"), cli_read.cmd);
91-
Cli.printf(PSTR("Number of arguments: %d\n"), (int)(cli_read.argc));
90+
Cli.printf("Command received: %s\n", cli_read.cmd);
91+
Cli.printf("Number of arguments: %d\n"), (int)(cli_read.argc);
9292
for(int i = 0; i < cli_read.argc; i++)
93-
Cli.printf(PSTR(" Argument %d: %s"), i, cli_read.argv[i]);
94-
Cli.printf(PSTR("\n"));
93+
Cli.printf(" Argument %d: %s", i, cli_read.argv[i]);
94+
Cli.printf("\n");
9595

9696
// Handle Commands
97-
if(strcmp(cli_read.cmd, PSTR("help")) == 0)
97+
if(strcmp(cli_read.cmd, "help") == 0)
9898
{
99-
Cli.printf(PSTR("Available Commands:\n"));
100-
Cli.printf(PSTR(" help - Current info.\n"));
101-
Cli.printf(PSTR(" led [on/off] - Turn LED ON or OFF.\n"));
102-
Cli.printf(PSTR(" version - Shows current firmware version.\n"));
99+
Cli.printf("Available Commands:\n");
100+
Cli.printf(" help - Current info.\n");
101+
Cli.printf(" led [on/off] - Turn LED ON or OFF.\n");
102+
Cli.printf(" version - Shows current firmware version.\n");
103103
}
104-
else if(strcmp(cli_read.cmd, PSTR("led")) == 0)
104+
else if(strcmp(cli_read.cmd, "led") == 0)
105105
{
106106
bool invalid_argv = false;
107107
char* led_mode = NULL;
@@ -112,30 +112,30 @@ void loop()
112112
else
113113
{
114114
led_mode = cli_read.argv[0];
115-
if(strcmp(led_mode, PSTR("on")) == 0)
115+
if(strcmp(led_mode, "on") == 0)
116116
{
117-
Cli.printf(PSTR("Turning LED ON.\n"));
117+
Cli.printf("Turning LED ON.\n");
118118
digitalWrite(COMMAND_LED, HIGH);
119119
}
120-
else if(strcmp(led_mode, PSTR("off")) == 0)
120+
else if(strcmp(led_mode, "off") == 0)
121121
{
122-
Cli.printf(PSTR("Turning LED OFF.\n"));
122+
Cli.printf("Turning LED OFF.\n");
123123
digitalWrite(COMMAND_LED, LOW);
124124
}
125125
else
126126
invalid_argv = true;
127127
}
128128

129129
if(invalid_argv)
130-
Cli.printf(PSTR("led command needs \"on\" or \"off\" arg.\n"));
130+
Cli.printf("led command needs \"on\" or \"off\" arg.\n");
131131
}
132-
else if(strcmp(cli_read.cmd, PSTR("version")) == 0)
132+
else if(strcmp(cli_read.cmd, "version") == 0)
133133
{
134-
Cli.printf(PSTR("FW App Version: %s\n"), FW_APP_VERSION);
134+
Cli.printf("FW App Version: %s\n", FW_APP_VERSION);
135135
}
136136
// ...
137137
else
138-
Cli.printf(PSTR("Unkown command.\n"));
139-
Cli.printf(PSTR("\n"));
138+
Cli.printf("Unkown command.\n");
139+
Cli.printf("\n");
140140
}
141141
}

examples/arduino/basic_usage_callbacks/src/main.cpp

Lines changed: 11 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -91,14 +91,14 @@ void setup()
9191
Cli.setup(&Serial, SERIAL_BAUDS);
9292

9393
// Add commands and bind callbacks to them
94-
Cli.add_cmd("led", &cmd_led, PSTR("led [on/off], Turn LED ON/OFF."));
95-
Cli.add_cmd("version", &cmd_version, PSTR("Shows firmware version."));
94+
Cli.add_cmd("led", &cmd_led, "led [on/off], Turn LED ON/OFF.");
95+
Cli.add_cmd("version", &cmd_version, "Shows firmware version.");
9696

9797
// The "help" command is already builtin and available from the CLI, and it
9898
// will shows added command descriptions, but you can setup a custom one
99-
Cli.add_cmd("help", &cmd_help, PSTR("Shows program help information."));
99+
Cli.add_cmd("help", &cmd_help, "Shows program help information.");
100100

101-
Cli.printf(PSTR("\nCommand Line Interface is ready\n\n"));
101+
Cli.printf("\nCommand Line Interface is ready\n\n");
102102
}
103103

104104
void loop()
@@ -117,8 +117,8 @@ void loop()
117117
void cmd_help(MINBASECLI* Cli, int argc, char* argv[])
118118
{
119119
// Show some Info text
120-
Cli->printf(PSTR("\nCustom Help Command\n"));
121-
Cli->printf(PSTR("MINBASECLI basic_usage_callbacks %s\n"), FW_APP_VERSION);
120+
Cli->printf("\nCustom Help Command\n");
121+
Cli->printf("MINBASECLI basic_usage_callbacks %s\n", FW_APP_VERSION);
122122

123123
// Call the builtin "help" function to show added command descriptions
124124
Cli->cmd_help(argc, argv);
@@ -135,25 +135,25 @@ void cmd_led(MINBASECLI* Cli, int argc, char* argv[])
135135
char* test_mode = argv[0];
136136
if(strcmp(test_mode, "on") == 0)
137137
{
138-
Cli->printf(PSTR("Turning LED ON.\n"));
138+
Cli->printf("Turning LED ON.\n");
139139
digitalWrite(COMMAND_LED, HIGH);
140140
}
141141
else if(strcmp(test_mode, "off") == 0)
142142
{
143-
Cli->printf(PSTR("Turning LED OFF.\n"));
143+
Cli->printf("Turning LED OFF.\n");
144144
digitalWrite(COMMAND_LED, LOW);
145145
}
146146
else
147147
{ invalid_argv = true; }
148148
}
149149

150150
if(invalid_argv)
151-
{ Cli->printf(PSTR("led command needs \"on\" or \"off\" arg.\n")); }
151+
{ Cli->printf("led command needs \"on\" or \"off\" arg.\n"); }
152152

153-
Cli->printf(PSTR("\n"));
153+
Cli->printf("\n");
154154
}
155155

156156
void cmd_version(MINBASECLI* Cli, int argc, char* argv[])
157157
{
158-
Cli->printf(PSTR("FW App Version: %s\n"), FW_APP_VERSION);
158+
Cli->printf("FW App Version: %s\n", FW_APP_VERSION);
159159
}

0 commit comments

Comments
 (0)