warning: deprecated conversion from string constant to 'char

[astrofrostbyte Profiel Privébericht]

Hieronder een klein stuk voorbeeld code die een warning geeft bij het doorgeven van een constant string aan een functie :

Code: Alles selecteren

#include <Wire.h>
#include <LiquidCrystal_I2C.h>

LiquidCrystal_I2C lcd(0x27, 20,4 );           // Create object , 20Characters , 4 lines

//-------------------------------------------------------------------------------------
void lcd_print(byte col, byte line, char *str)
{
  lcd.setCursor(col,line);
  lcd.print(str);
}
//-------------------------------------------------------------------------------------
void setup()
{ Serial.begin(9600);  Serial.print("Startup testcode");

  lcd.init();
  lcd.backlight();
  lcd.print("Startup Testcode");

  lcd_print( 0, 1, "functiontest");                         // <<----   Boosdoener !
}//setup
//-------------------------------------------------------------------------------------
void loop() {}

Geeft de volgende warning :
sketch_jun09b.ino: In function 'void setup()':
sketch_jun09b.ino:14: warning: deprecated conversion from string constant to 'char*'

De boosdoener is :

Code: Alles selecteren

 lcd_print( 0, 1, "functiontest");                         // <<----   Boosdoener !

Is er een betere methode on dit in C te omschrijven ??

Ik kan het gewoon aanpassen door de constant te casten, en de warning is nu weg.

Code: Alles selecteren

lcd_print( 0, 1, (char *)"functiontest");

Maar ik twijfel of dit de juiste manier van aanpak is.

[Karel Profiel Privébericht]

Ik vermoed dat het probleem erin zit dat de aangeroepen functie lcd_print niet "weet" dat de meegegeven parameter in string constant is. Als lcd_print de variabel str zou willen wijzigen kan dat. Dan zit je echter in vaste data te schrijven. Als je dan een te lange string zou toekennen overschrijf je geheugen dat voor wat anders wordt gebruikt en wordt het resultaat onvoorspelbaar.

Je zou eens in de source van lcd.print moeten kijken wat daarin anders wordt gedaan, zodat je de warning niet krijgt als je lcd.print aanroept.

[Karel Profiel Privébericht]

Of je declareert

Code: Alles selecteren

void lcd_print(byte col, byte line, const str[])


[astrofrostbyte Profiel Privébericht]

thanks, in de lcd library gebruiken ze inderdaad hetzelfde zoals jij ook aangeeft :

void LiquidCrystal_I2C::printstr(const char c[] ) {

[Karel Profiel Privébericht]

Of je declareert

Code: Alles selecteren

void lcd_print(byte col, byte line, const str[])


Zit een foutje in. moet zijn:

Code: Alles selecteren

void lcd_print(byte col, byte line, const char str[])


@Moderator: waarom mag ik een eigen bericht niet aanpassen? Zo blijft een fout zichtbaar, en bestaat de kans dat iemand na het lezen van de (foute) oplossing foute code overneemt. Die vraag had ik graag in het bericht zelf aangepast. (met omschrijving van reden tot wijziging).

Groet, CJ