[Gelöst] [skinflatplus] warning: ‘<unnamed>.cSimpleContent::ContentType’ may be used uninitialized [-Wmaybe-uninitialized]

MegaV0lt · 6. Juli 2023

Ich versuche krampfhaft im Netz was zu finden, um die Meldung

Code

In file included from complexcontent.c:1:
In copy constructor ‘cSimpleContent::cSimpleContent(const cSimpleContent&)’,
    inlined from ‘void __gnu_cxx::new_allocator<_Tp>::construct(_Up*, _Args&& ...) [with _Up = cSimpleContent; _Args = {cSimpleContent}; _Tp = cSimpleContent]’ at /usr/include/c++/11/ext/new_allocator.h:162:4,
    inlined from ‘static void std::allocator_traits<std::allocator<_CharT> >::construct(std::allocator_traits<std::allocator<_CharT> >::allocator_type&, _Up*, _Args&& ...) [with _Up = cSimpleContent; _Args = {cSimpleContent}; _Tp = cSimpleContent]’ at /usr/include/c++/11/bits/alloc_traits.h:516:17,
    inlined from ‘void std::vector<_Tp, _Alloc>::emplace_back(_Args&& ...) [with _Args = {cSimpleContent}; _Tp = cSimpleContent; _Alloc = std::allocator<cSimpleContent>]’ at /usr/include/c++/11/bits/vector.tcc:115:30,
    inlined from ‘void std::vector<_Tp, _Alloc>::push_back(std::vector<_Tp, _Alloc>::value_type&&) [with _Tp = cSimpleContent; _Alloc = std::allocator<cSimpleContent>]’ at /usr/include/c++/11/bits/stl_vector.h:1204:21,
    inlined from ‘void cComplexContent::AddText(const char*, bool, cRect, tColor, tColor, cFont*, int, int, int)’ at complexcontent.c:153:21:
complexcontent.h:19:7: warning: ‘<unnamed>.cSimpleContent::ContentType’ may be used uninitialized [-Wmaybe-uninitialized]
   19 | class cSimpleContent
      |       ^~~~~~~~~~~~~~
complexcontent.c: In member function ‘void cComplexContent::AddText(const char*, bool, cRect, tColor, tColor, cFont*, int, int, int)’:
complexcontent.c:153:37: note: ‘<anonymous>’ declared here
  153 |   Contents.push_back(cSimpleContent());
      |

Alles anzeigen

weg zu bekommen. Ich hab keine Ahnung wo ich da ansetzen muss. Leider kenn ich mich mit Programmieren nicht aus. Mit mühe und Not kann ich kleinere Änderungen einbauen. Bei anderen Skin's habe ich mich schon umgeschaut, aber nichts gefunden. Das mit dem ComplexContent scheint nur flatPlus zu haben…

Das komplette Buildlog ist hier: https://launchpadlibrarian.net…dr0~jammy_BUILDING.txt.gz

Den Fehler:

Code

warning: deleting object of polymorphic class type ‘cTextScroll’ which has non-virtual destructor might cause undefined behavior [-Wdelete-non-virtual-dtor]

konnte ich schon lösen. Vielleicht hat ja jemand ne Idee? GIT in der Signatur

wirbel · 6. Juli 2023

Ich mag nicht detailliert nachsehen, aber für mich bedeutet es, es gibt ein

Code

cSimpleContent::cSimpleContent(const cSimpleContent&)

in dem ein Member namens

Code

ContentType

vergessen wurde zuzuweisen. Da es ein copy constructor ist, muss die Variable wahrscheinlich vom Argument

Code

(const cSimpleContent&)

kommen. Der Name dieser Variable ist dem compiler ja recht egal und taucht in der Meldung nicht auf..

MegaV0lt · 6. Juli 2023

In der complexcontent.h:

Code

class cComplexContent
{
private:
    std::vector<cSimpleContent> Contents;

    cPixmap *Pixmap, *PixmapImage;
    cRect Position;

    tColor ColorBg;

    bool FullFillBackground;
    int DrawPortHeight;
    int ScrollSize;
    bool isShown;
    bool isScrollingActive;

    cOsd *Osd;

    void CalculateDrawPortHeight(void);
    virtual cPixmap *CreatePixmap(int Layer, const cRect &ViewPort, const cRect &DrawPort = cRect::Null);

Alles anzeigen

Würde das gehen?:

Code

std::vector<cSimpleContent> Contents {};

oder irgendwo in complexcontent.c?

FireFly · 6. Juli 2023

Ich würde sagen, man kann den ContentType abfragen

int GetContentType(void) { return ContentType; }

ohne dass im Konstruktor ein Wert gesetzt wurde:

Code

public:
    cSimpleContent(void) {
        Image = NULL;
        Font = NULL;
    }

MegaV0lt · 7. Juli 2023

Vielen Dank, aber ich kapier das einfach nicht.

FireFly · 7. Juli 2023

Füge in complexcontent.h hinter Font = NULL; als Zeile 35 ContentType = CT_TEXT; ein und schaue, ob es damit compiliert. Damit ist der Default ContentType CT_TEXT.

wirbel · 7. Juli 2023

Und nach dieser Zeile bitte ebenso default Werte zuweisen für

int TextWidth, TextHeight, TextAlignment;

tColor ColorFg, ColorBg;

Das beseitigt dann wahrscheinlich die Warnung, aber nicht das Problem als solches.

Die Klasse cSimpleContent wird in std::vector benutzt, deswegen muss sie zwei Bedingungen erfüllen: CopyAssignable and CopyConstructible.

Den copy constructor gibt es noch nicht in der Klasse, gcc fügt einen ein, aber der ist anscheinend nicht komplett:

"If no user-defined copy constructors are provided for a class type (struct, class, or union), the compiler will always declare a copy constructor as a non-explicit inline public member of its class. This implicitly-declared copy constructor has the form T::T(const T&)"

mit T = cSimpleContent

Die beste Idee wäre, zusätzlich einen copy constructor anzulegen für cSimpleContent. Vielleicht so, rhs steht für 'right hand side'

Code

    cSimpleContent(const cSimpleContent& rhs) {
        ContentType = rhs.ContentType;
        Position = rhs.Position;
        TextWidth = rhs.TextWidth;
        TextHeight = rhs.TextHeight;
        TextAlignment = rhs.TextAlignment;
        ColorFg = rhs.ColorFg;
        ColorBg = rhs.ColorBg;
        Text = rhs.Text;
        Image = rhs.Image;
        Font = rhs.Font;
    }

Alles anzeigen

MegaV0lt · 7. Juli 2023

Danke für die Hinweise. Ich probier das aus, wenn ich Montag am VDR bin.

Was kann ich als Vorgabe bei tColor nehmen? 0 oder NULL

Und wo muss der copy constructor rein? Ab Zeile 44 in der .h steht fast das gleiche aber mit other statt rhs

Auszug:

Code

class cSimpleContent
{
private:
    int ContentType;
    cRect Position;

    int TextWidth, TextHeight, TextAlignment;
    tColor ColorFg, ColorBg;
    std::string Text;
    cImage *Image;
    cFont *Font;

public:
    cSimpleContent(void) {
        Image = NULL;
        Font = NULL;
       
        ContentType = CT_TEXT;  // Added to avoid compiler warning
        int TextWidth = 0, TextHeight = 0, TextAlignment = 0;
        //tColor ColorFg, ColorBg; // TODO ???
    }

    ~cSimpleContent() {
    }

    cSimpleContent& operator=(const cSimpleContent& other) { // Zeile 44 complexcontent.h
        if( this != &other ) {
            this->ContentType = other.ContentType;
            this->Position = other.Position;
            this->Text = other.Text;
            this->TextWidth = other.TextWidth;
            this->TextHeight = other.TextHeight;
            this->TextAlignment = other.TextAlignment;
            this->ColorFg = other.ColorFg;
            this->ColorBg = other.ColorBg;
            this->Image = other.Image;
            this->Font = other.Font;
        }
        return *this;
    }

    void SetText(const char *text, bool Multiline, cRect position, tColor colorFg, tColor colorBg, cFont *font, int textWidth = 0, int textHeight = 0, int textAlignment = taDefault) {
        ContentType = CT_Text;
        Text = text;

        Font = font;
        Position = position;

        if( Multiline ) {
            ContentType = CT_TextMultiline;
        }
        ColorFg = colorFg; ColorBg = colorBg;
        TextWidth = textWidth; TextHeight = textHeight; TextAlignment = textAlignment;
    }

    void SetImage(cImage *image, cRect position) {
        ContentType = CT_Image;
        Image = image;
        Position = position;
    }
...

Alles anzeigen

wirbel · 7. Juli 2023

Am besten vor

Code

 ~cSimpleContent() {

PS: 'other' anstelle von rhs wäre völlig ok. Ich mag halt Bezeichnungen, die ich bei anderen Programmierern wieder finde und verstehe (oder glaube das zu tun.. ).

MegaV0lt · 7. Juli 2023

Danke sehr!

Ich baue das ein und teste am Montag

MegaV0lt · 10. Juli 2023

Folgender Stand:

Plugin baut mit dem 'copy constructor' von wirbel und dem Vorschlag von FireFly

Das Zuweisen von den anderen Werten führt leider zu Warnungen:

Code

In file included from baserender.h:7,
                 from displaychannel.h:4,
                 from imagecache.c:5:
complexcontent.h: In constructor ‘cSimpleContent::cSimpleContent()’:
complexcontent.h:37:13: warning: unused variable ‘TextWidth’ [-Wunused-variable]
   37 |         int TextWidth = 0, TextHeight = 0, TextAlignment = 0;
      |             ^~~~~~~~~
complexcontent.h:37:28: warning: unused variable ‘TextHeight’ [-Wunused-variable]
   37 |         int TextWidth = 0, TextHeight = 0, TextAlignment = 0;
      |                            ^~~~~~~~~~
complexcontent.h:37:44: warning: unused variable ‘TextAlignment’ [-Wunused-variable]
   37 |         int TextWidth = 0, TextHeight = 0, TextAlignment = 0;
      |                                            ^~~~~~~~~~~~~

Alles anzeigen

Wiederholt sich ein paar mal

Die Datei sieht so aus: https://www.dropbox.com/s/05ub…nj7/complexcontent.h?dl=0 (Dropbox)

Änderungen in Zeile 36 und der Block ab Zeile 41-52

FireFly · 10. Juli 2023

Du musst in Zeile 37 das "int" vor "TextWidth = 0, ..." weglassen, weil damit neue lokale Variablen definiert werden (die danach nicht benutzt werden) und nicht die Membervariablen der Klasse initialisiert werden.

Um am besten noch in dem enum ab Zeile 8 ein CT_NONE einführen und damit in Zeile 36 initialisieren anstatt mit CT_TEXT. Dann musst Du aber auch überall, wo ContentType geprüft wird, kontrollieren, dass er das nur die anderen CT_* bearbeitet.

MegaV0lt · 10. Juli 2023

Super! Baut nun ohne Warnungen. Test in Virtualbox sieht auch gut aus.

Vielen Dank für die Unterstützung wirbel und FireFly

[Gelöst] [skinflatplus] warning: ‘<unnamed>.cSimpleContent::ContentType’ may be used uninitialized [-Wmaybe-uninitialized]

MegaV0lt 10. Juli 2023

Jetzt mitmachen!

Teilen

Tags

Benutzer online in diesem Thema