Can't compile TestShape example

[GeorgeFlorian]

Hello !

As the title says I can't compile the testshape example.

/* Derived from Adafruit RGB_matrix_Panel library */

#include <Adafruit_GFX.h>   // Core graphics library
#include <P3RGB64x32MatrixPanel.h>

// constructor with default pin wiring
P3RGB64x32MatrixPanel matrix;

// use this constructor for custom pin wiring instead of the default above
// these pins are an example, you may modify this according to your needs
//P3RGB64x32MatrixPanel matrix(25, 26, 27, 21, 22, 23, 15, 32, 33, 12, 16, 17, 18);

void setup() {

  matrix.begin();

  // draw a pixel in solid white
  matrix.drawPixel(0, 0, matrix.color444(15, 15, 15)); 
  delay(500);

  // fix the screen with green
  matrix.fillRect(0, 0, matrix.width(), matrix.height(), matrix.color444(0, 15, 0));
  delay(500);

  // draw a box in yellow
  matrix.drawRect(0, 0, matrix.width(), matrix.height(), matrix.color444(15, 15, 0));
  delay(500);

  // draw an 'X' in red
  matrix.drawLine(0, 0, matrix.width()-1, matrix.height()-1, matrix.color444(15, 0, 0));
  matrix.drawLine(matrix.width()-1, 0, 0, matrix.height()-1, matrix.color444(15, 0, 0));
  delay(500);

  // draw a blue circle
  matrix.drawCircle(10, 10, 10, matrix.color444(0, 0, 15));
  delay(500);

  // fill a violet circle
  matrix.fillCircle(40, 21, 10, matrix.color444(15, 0, 15));
  delay(500);

  // fill the screen with 'black'
  matrix.fillScreen(matrix.color444(0, 0, 0));

  // draw some text!
  matrix.setTextSize(1);     // size 1 == 8 pixels high
  matrix.setTextWrap(false); // Don't wrap at end of line - will do ourselves

  matrix.setCursor(8, 0);    // start at top left, with 8 pixel of spacing
  uint8_t w = 0;
  char *str = "P3indoorSMDDisplay";
  for (w=0; w<8; w++) {
    matrix.setTextColor(Wheel(w));
    matrix.print(str[w]);
  }
  matrix.setCursor(2, 8);    // next line
  for (w=8; w<18; w++) {
    matrix.setTextColor(Wheel(w));
    matrix.print(str[w]);
  }
  matrix.println();
  //matrix.setTextColor(matrix.Color333(4,4,4));
  //matrix.println("Industries");
  matrix.setTextColor(matrix.color444(15,15,15));
  matrix.println("LED MATRIX!");

  // print each letter with a rainbow color
  matrix.setTextColor(matrix.color444(15,0,0));
  matrix.print('3');
  matrix.setTextColor(matrix.color444(15,4,0)); 
  matrix.print('2');
  matrix.setTextColor(matrix.color444(15,15,0));
  matrix.print('x');
  matrix.setTextColor(matrix.color444(8,15,0)); 
  matrix.print('6');
  matrix.setTextColor(matrix.color444(0,15,0));  
  matrix.print('4');
  matrix.setCursor(34, 24);  
  matrix.setTextColor(matrix.color444(0,15,15)); 
  matrix.print("*");
  matrix.setTextColor(matrix.color444(0,8,15)); 
  matrix.print('R');
  matrix.setTextColor(matrix.color444(0,0,15));
  matrix.print('G');
  matrix.setTextColor(matrix.color444(8,0,15)); 
  matrix.print("B");
  matrix.setTextColor(matrix.color444(15,0,8)); 
  matrix.println("*");

  // whew!
}

void loop() {
  // do nothing
}

// Input a value 0 to 24 to get a color value.
// The colours are a transition r - g - b - back to r.
uint16_t Wheel(byte WheelPos) {
  if(WheelPos < 8) {
   return matrix.color444(15 - WheelPos*2, WheelPos*2, 0);
  } else if(WheelPos < 16) {
   WheelPos -= 8;
   return matrix.color444(0, 15-WheelPos*2, WheelPos*2);
  } else {
   WheelPos -= 16;
   return matrix.color444(0, WheelPos*2, 7 - WheelPos*2);
  }
}

Here is the output:

compilation terminated.
*** [.pioenvs/esp32doit-devkit-v1/libfd9/Adafruit GFX Library_ID13/Adafruit_SPITFT.cpp.o] Error 1
src/main.cpp: In function 'void setup()':
src/main.cpp:51:15: warning: ISO C++ forbids converting a string constant to 'char*' [-Wwrite-strings]
char *str = "P3indoorSMDDisplay";
^
src/main.cpp:53:32: error: 'Wheel' was not declared in this scope
matrix.setTextColor(Wheel(w));
^
src/main.cpp:58:32: error: 'Wheel' was not declared in this scope
matrix.setTextColor(Wheel(w));
^
*** [.pioenvs/esp32doit-devkit-v1/src/main.cpp.o] Error 1

[NeoCat]

How did you setup the Arduino IDE?

[GeorgeFlorian]

How did you setup the Arduino IDE?

Like this and it works just fine with other sketches.

https://i.postimg.cc/VkT3rG5t/arduino.png

I've installed both libraries as it says in the Readme.

[NeoCat]

What happens if move definition of Wheel() to the top of the source?

[GeorgeFlorian]

What happens if move definition of Wheel() to the top of the source?

Wheel is now defined but there still are compiler errors:

compilation terminated.
Compiling .pioenvs/esp32doit-devkit-v1/FrameworkArduino/FunctionalInterrupt.cpp.o
Compiling .pioenvs/esp32doit-devkit-v1/FrameworkArduino/HardwareSerial.cpp.o
*** [.pioenvs/esp32doit-devkit-v1/libfd9/Adafruit GFX Library_ID13/Adafruit_SPITFT.cpp.o] Error 1
src/main.cpp: In function 'void setup()':
src/main.cpp:64:15: warning: ISO C++ forbids converting a string constant to 'char*' [-Wwrite-strings]
char *str = "P3indoorSMDDisplay";
^

[NeoCat]

Adding const at the top of the line may solve that warning.

[GeorgeFlorian]

Adding const at the top of the line may solve that warning.

You mean const uint16_t Wheel(byte WheelPos) ? Because it doesn't work.

[NeoCat]

Of course not. const char *str = "...";

[GeorgeFlorian]

Of course not. const char *str = "...";

That's way better. The only error now is this:

compilation terminated.
Compiling .pioenvs/esp32doit-devkit-v1/FrameworkArduino/MD5Builder.cpp.o
*** [.pioenvs/esp32doit-devkit-v1/libfd9/Adafruit GFX Library_ID13/Adafruit_SPITFT.cpp.o] Error 1

[NeoCat]

Heee, error with no messages... Maybe you need to clean up your environment and recompile everything...

[GeorgeFlorian]

Heee, error with no messages... Maybe you need to clean up your environment and recompile everything...

Thank you for your assistance !

The code compiles on Arduino IDE. The problem is with PlatformIO IDE.

I will post this error on the Adafruit GFX github.

I will close this now as it doesn't seem to be an issue from your library.