deprecated conversion

[Georg M. (g_m)]

Demo Code von Microchip Technology Incorporated:

/* ----------------------------------------------------------------
Demo Code
This code example is used to continuously send the 'Hello World!' string through USART. A string is sent character by character. The ‘USART1_sendString’ function calls the ‘USART1_sendChar’ function for each character in the ‘Hello Word!’ string. Before sending each character, the ‘USART1_sendChar’ function waits for the previous character transmission to be completed. This is done by polling the Data Register Empty Interrupt Flag, DREIF, from the STATUS register until it is set.
------------------------------------------------------------------*/
#define F_CPU 3333333
#define USART1_BAUD_RATE(BAUD_RATE) ((float)(F_CPU * 64 / (16 * (float)BAUD_RATE)) + 0.5)
#include <avr/io.h>
#include <util/delay.h>
#include <string.h>
void USART1_init(void);
void USART1_sendChar(char c);
void USART1_sendString(char *str);
void USART1_init(void)
{
 PORTC.DIR &= ~PIN1_bm;
 PORTC.DIR |= PIN0_bm;
 
 USART1.BAUD = (uint16_t)USART1_BAUD_RATE(9600);
 USART1.CTRLB |= USART_TXEN_bm;
}
void USART1_sendChar(char c)
{
 while (!(USART1.STATUS & USART_DREIF_bm))
 {
  ;
 } 
 USART1.TXDATAL = c;
}
void USART1_sendString(char *str)
{
 for(size_t i = 0; i < strlen(str); i++)
 {
  USART1_sendChar(str[i]);
 }
}
int main(void)
{
 USART1_init();
 
 while (1)
 {
  USART1_sendString("Hello World!\r\n");
  _delay_ms(500);
 }
}

https://ww1.microchip.com/downloads/en/Appnotes/TB3216-Getting-Started-with-USART-DS90003216.pdf

Der Compiler von Microchip Technology Incorporated bemängelt aber die 
Zeile

 USART1_sendString("Hello World!\r\n");

mit

Warning: deprecated conversion from string constant to 'char*' [-Wwrite-strings]

Wie wäre es dann richtiger?

[Mombert H. (mh_mh)]

sendString braucht offensichtlich ein const also: char const*

[Niklas G. (erlkoenig)]

Richtig wäre es, wenn "USART1_sendString" ein Argument vom Typ "const 
char*" übernehmen würde. Schließlich sollte es den Text nicht verändern.

void USART1_sendString(const char *str)
{
 for(size_t i = 0; i < strlen(str); i++)
 {
  USART1_sendChar(str[i]);
 }
}

Allerdings wird hier strlen in jedem Durchlauf aufgerufen, d.h. super 
ineffizient. Viel besser geht es so:

void USART1_sendString(const char *str)
{
 for(const char* p = str; *p != '\0'; ++p)
 {
  USART1_sendChar(*p);
 }
}

Weil es ja um AVR geht, wäre es natürlich noch besser den String nur im 
Flash abzulegen.

Falls du mal den Fall hast dass du eine Funktion wie USART1_sendChar 
hast die den String tatsächlich ändert, muss man eine Kopie des Strings 
im RAM (hier: auf dem Stack) anlegen:

char str [] = "Hello World!\r\n";
USART1_sendString(str);

Hier ist also wieder schön zu sehen, dass die Mikrocontroller-Hersteller 
desaströse Code-Qualität produzieren, und Microchip ist ganz vorne 
dabei.

[Mombert H. (mh_mh)]

Niklas G. schrieb:
> Allerdings wird hier strlen in jedem Durchlauf aufgerufen, d.h. super
> ineffizient.

Ab welcher Stringlänge fängt das an ineffizient zu werden, wenn die 
Baudrate 9600 ist?

Niklas G. schrieb:
> Hier ist also wieder schön zu sehen, dass die Mikrocontroller-Hersteller
> desaströse Code-Qualität produzieren, und Microchip ist ganz vorne
> dabei.
Das ist bei diesem Beispiel etwas übertrieben.