32F746Disco, SW4STM32, kompiliert, nach kleiner Änderung aber nicht.

[J. T. (chaoskind)]

MoinMoin,

ich spiele gerade ein wenig mit meinem Disco-Board rum. Das Projekt mit 
Cube erstellt, ein paar bunte Zeilen auf dem Display anzeigen lassen. 
Klappt soweit alles.

Wenn ich dann aber die Farben über die Zeit ändern möchte, bekommen die 
ganzen Cube-Funktionen die Fehlermeldung "invalid storage class for 
function..."

Den Teil nachdem es nicht mehr klappt, habe ich mal extra weit 
eingerückt. Wenn ich den Teil weglasse, kompiliert alles wunderbar, 
nehme ich ihn mit rein, gehts nicht mehr...

Evtl sieht ja jemand von euch, was daran nicht stimmt? Der angesprochene 
Teil kommt in der while-schleife von der main.

int main(void)
{
  /* USER CODE BEGIN 1 */

  uint8_t Ringpuffer[] = {"                              Dies ist ein Scrolltext. Er wandert von selbst das Display entlang              "};
  uint8_t Lesekopf = 0;
  uint8_t Schreibkopf = 0;
  uint8_t i = 0;
  uint8_t j = 0;

  uint8_t Farbflags = 0b00001100;        //bit7,6 = X, bit5 = Rot(not running/running), bit4 = Rot(down/up), bit3 = Gruen(nr/r), bit2 = Gruen (d/u), bit1 = Blau(nr/r), bit0 = Blau(d/u)
  uint8_t Rot = 255;
  uint8_t Gruen = 0;
  uint8_t Blau = 0;
  uint32_t Farbe = 0;

  uint8_t Textpuffer[sizeof(Ringpuffer)] = {0};

  uint16_t k = 100;

  /* USER CODE END 1 */

  /* Enable I-Cache-------------------------------------------------------------*/
  SCB_EnableICache();

  /* Enable D-Cache-------------------------------------------------------------*/
  SCB_EnableDCache();

  /* MCU Configuration----------------------------------------------------------*/

  /* Reset of all peripherals, Initializes the Flash interface and the Systick. */
  HAL_Init();

  /* USER CODE BEGIN Init */

  /* USER CODE END Init */

  /* Configure the system clock */
  SystemClock_Config();

  /* USER CODE BEGIN SysInit */

  /* USER CODE END SysInit */

  /* Initialize all configured peripherals */
  MX_GPIO_Init();
  MX_USART6_UART_Init();
  MX_SPI2_Init();
  MX_RNG_Init();
  MX_QUADSPI_Init();
  MX_LTDC_Init();
  MX_FMC_Init();
  MX_DMA2D_Init();
  MX_CRC_Init();
  MX_ADC3_Init();
  /* USER CODE BEGIN 2 */

  BSP_LCD_Init();
  BSP_LCD_Init();
  BSP_LCD_LayerDefaultInit(0, LCD_FB_START_ADDRESS);
  BSP_LCD_LayerDefaultInit(  1, ( LCD_FB_START_ADDRESS + (4*1024*1024) )  );
  BSP_LCD_SelectLayer(0);
  BSP_LCD_SetTextColor(0xFFFFFFFF);
  BSP_LCD_SetBackColor(0x00000000);
  BSP_LCD_Clear(0x00000000);
  BSP_LCD_SelectLayer(1);
  BSP_LCD_SetBackColor(0x00000000);
  BSP_LCD_Clear(0x00000000);
  BSP_LCD_SetTextColor(0xFFFF0000);
  BSP_LCD_DisplayStringAtLine(0, "Line0");
  BSP_LCD_SetTextColor(0xFFFF8800);
  BSP_LCD_DisplayStringAtLine(1, "Line1");
  BSP_LCD_SetTextColor(0xFFFFFF00);
  BSP_LCD_DisplayStringAtLine(2, "Line2");
  BSP_LCD_SetTextColor(0xFF88FF00);
  BSP_LCD_DisplayStringAtLine(3, "Line3");
  BSP_LCD_SetTextColor(0xFF00FF00);
  BSP_LCD_DisplayStringAtLine(4, "Line4");
  BSP_LCD_SetTextColor(0xFF00FF88);
  BSP_LCD_DisplayStringAtLine(5, "Line5");
  BSP_LCD_SetTextColor(0xFF00FFFF);
  BSP_LCD_DisplayStringAtLine(6, "Line6");
  BSP_LCD_SetTextColor(0xFF0088FF);
  BSP_LCD_DisplayStringAtLine(7, "Line7");
  BSP_LCD_SetTextColor(0xFF0000FF);
  BSP_LCD_DisplayStringAtLine(8, "Line8");
  BSP_LCD_SetTextColor(0xFF8800FF);
  BSP_LCD_DisplayStringAtLine(9, "Line9");
  BSP_LCD_SetTextColor(0xFFFF00FF);
  BSP_LCD_DisplayStringAtLine(10, "Line10");

  HAL_ADC_Init(&hadc3);
  HAL_ADC_Start(&hadc3);

  HAL_RNG_Init(&hrng);

  BSP_LCD_SetTextColor(0xFFFF0088);
  /* USER CODE END 2 */

  /* Infinite loop */
  /* USER CODE BEGIN WHILE */
  while (1)
  {
    Lesekopf = j;
    j++;
    j %= sizeof(Ringpuffer);
    i = 0;

    while( i < sizeof(Ringpuffer) )
    {
            if( (Farbflags & (1 << Rot_run)) != 0)
            {
              if ( (Farbflags & (1 << Rot_dir)) == 0)
              {
                Rot--;
                if(Rot == 0)
                {
                  Farbflags &= ~(1 << Rot_run);
                  Farbflags |= (1 << Rot_dir);
                  Farbflags |= (1 << Gruen_run);
                }
              }
              else
              {
                Rot++;
                if(Rot == 255)
                {
                  Farbflags &= ~(1 << Rot_run);
                  Farbflags |= (1 << Rot_dir);
                  Farbflags |= (1 << Gruen_run);
                }
              }

            if( (Farbflags & (1 << Gruen_run)) != 0)
            {
              if ( (Farbflags & (1 << Gruen_run)) == 0)
              {
                Gruen--;
                if(Gruen == 0)
                {
                  Farbflags &= ~(1 << Gruen_run);
                  Farbflags |= (1 << Gruen_dir);
                  Farbflags |= (1 << Blau_run);
                }
              }
              else
              {
                Gruen++;
                if(Gruen == 255)
                {
                  Farbflags &= ~(1 << Gruen_run);
                  Farbflags |= (1 << Gruen_dir);
                  Farbflags |= (1 << Blau_run);
                }
              }
            }

            if( (Farbflags & (1 << Blau_run)) != 0)
            {
              if ( (Farbflags & (1 << Blau_run)) == 0)
              {
                Blau--;
                if(Blau == 0)
                {
                  Farbflags &= ~(1 << Blau_run);
                  Farbflags |= (1 << Blau_dir);
                  Farbflags |= (1 << Rot_run);
                }
              }
              else
              {
                Blau++;
                if(Blau == 255)
                {
                  Farbflags &= ~(1 << Blau_run);
                  Farbflags |= (1 << Blau_dir);
                  Farbflags |= (1 << Rot_run);
                }
              }
            }


      Textpuffer[i] = Ringpuffer[Lesekopf];
      i++;
      Lesekopf++;
      Lesekopf %= sizeof(Ringpuffer);
    }

    BSP_LCD_DisplayStringAtLine(0, &Textpuffer[0]);
    HAL_Delay(k);

  /* USER CODE END WHILE */

  /* USER CODE BEGIN 3 */

  }
  /* USER CODE END 3 */

}

Ich erzeuge ja keine neuen Variablen oder ähnliches, ich lasse einfach 
nur den weit eingerückten Teil weg. Das dürfte doch eigentlich am 
Compileverhalten gar nichts ändern?

MfG Chaos

[Stefan F. (Gast)]

Wie lautet die vollständige Fehlermeldung des Compilers?

[Rufus Τ. F. (rufus)]

J. T. schrieb:
> ich lasse einfach nur den weit eingerückten Teil weg.

Zeig Deinen modifizierten Code. Bitte nicht im Text, sondern als 
Attachment.

[J. T. (chaoskind)]

Die Änderung ist Zeile 210 bis Zeile 279. Lasse ich diese Zeilen weg, 
geht es.

Stefanus F. schrieb:
> Wie lautet die vollständige Fehlermeldung des Compilers?

../Src/main.c:708:13: error: invalid storage class for function 
'MX_GPIO_Init'
 static void MX_GPIO_Init(void)
             ^~~~~~~~~~~~
../Src/main.c:1066:1: error: expected declaration or statement at end of 
input
 }


irgendwo muss da also ne geschweifte Klammer zuviel oder zuwenig sein. 
Aber jedesmal wenn ich drüber schau, finde ich soviele öffende, wie 
schließende.

Und noch weiter geschuat, Zeile 1066 ist die schließende Klammer vom

void _Error_Handler(char *file, int line)
{
  /* USER CODE BEGIN Error_Handler_Debug */
  /* User can add his own implementation to report the HAL error return 
state */
  while(1)
  {
  }
  /* USER CODE END Error_Handler_Debug */
}

[Harry L. (mysth)]

Erstmal ein Tip:

Im Eclipse den Code komplett markieren und <STRG> <SHIFT> F drücken.
Damit wird der Code formatiert und autom. korrekt eingerückt.
Dann erkennt man auch sofort, daß in deiner while-Schleie die 
schliessende Klammer fehlt.

Das Ende sollte dann so aussehen:

        Textpuffer[i] = Ringpuffer[Lesekopf];
        i++;
        Lesekopf++;
        Lesekopf %= sizeof(Ringpuffer);
      }

    BSP_LCD_DisplayStringAtLine (0, &Textpuffer[0]);
    HAL_Delay (k);
  }

      /* USER CODE END WHILE */

[J. T. (chaoskind)]

Harry L. schrieb:
> Im Eclipse den Code komplett markieren und <STRG> <SHIFT> F drücken.

Prinzipiell ein guter Tip, kann man das Autoformat auch so einstellen 
dass er statt

bla(){
}

auch so:

bla
{

}

einrückt?

Und damit würden dann ja auch die extra zur Markierung eingerückten 
Stellen verloren gehen... Wobei das könnte man ja nochmal machen, 
danach.

Und wenn ich dann die schließende Klammer nach dem HAL_Delay(k) einfüge, 
kompiliert er auch wieder. Ich verstehe nur nicht ganz, wieso die dann 
dahin muss. An der Stelle mache ich gar keine Veränderung, wenn ich den 
extra eingerückten Teil ändere? ~mit verwirrtem Blick...

[J. T. (chaoskind)]

P.S.

Das war doch nicht die Stelle. Ich hatte eine schließende in der Abfrage 
nach den Rot-Flags vergessen....

Danke für eure Hilfe

[Harry L. (mysth)]

Ja, den Formatierungs-.Stil kannst du unter

Window->Prferences->C/C++->Code Style->Formatter
ändern.

[J. T. (chaoskind)]

Harry L. schrieb:
> Ja, den Formatierungs-.Stil kannst du unter
>
> Window->Prferences->C/C++->Code Style->Formatter
> ändern.

Erstmal danke für die Info. Ich verstehe die Beispiele aber ehrlich 
gesagt nicht so ganz. Ich hab nun den Allman Style gewählt, der sah 
eigentlich so aus, wie ich es mir gewünscht hab. Jedoch macht er jetzt 
die Initialisierung der Arrays über mehrere Zeilen. Weißt du auch dafür 
Abhilfe?

uint8_t Textpuffer[sizeof(Ringpuffer)] =
  { 0 };

^^da geht das ja noch halbwegs, aber bei den Texten sieht das dann 
irgendwann komisch aus...

uint8_t Ringpuffer[] =
      {
          "                              Dies ist ein Scrolltext. Er wandert von selbst das Display entlang. Nebenbei bemerkt, es hat sich auch ein automatischer Farbwechsel eingeschlichen ;-)              " };

Ich hätte gern, dass er alles in eine Zeile packt

[Harry L. (mysth)]

J. T. schrieb:
> Weißt du auch dafür
> Abhilfe?

Leider nein.
Ich bin da sehr pragmatisch.
Ich verwende "GNU (built in)"

Das kommt meinen Vorstellungen recht nahe. (wenn auch nicht perfekt)

Mit der detaillierten Konfiguration hab ich mich nie beschäftigt.

Ich denke auch nicht, daß sowas zielführend ist, wenn jeder seinen 
eigenen Code-Style entwickelt.

[pegel (Gast)]

J. T. schrieb:
> Ich hätte gern, dass er alles in eine Zeile packt

Schau mal unter Window -> Preferences und gib im Suchfeld wrap ein.