[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-bugs-dist
Subject:    [Bug 303561] New: Plasma::ToolButton breaks compilation with some external libraries due to use slot
From:       Michał D. (Emdek) <emdeck () gmail ! com>
Date:       2012-07-15 14:16:46
Message-ID: bug-303561-17878 () http ! bugs ! kde ! org/
[Download RAW message or body]

https://bugs.kde.org/show_bug.cgi?id=303561

            Bug ID: 303561
          Severity: major
           Version: 4.8.4
          Priority: NOR
          Assignee: plasma-bugs@kde.org
           Summary: Plasma::ToolButton breaks compilation with some
                    external libraries due to use slots keyword instead of
                    Q_SLOTS
    Classification: Unclassified
                OS: Linux
          Reporter: emdeck@gmail.com
          Hardware: Ubuntu Packages
            Status: UNCONFIRMED
         Component: general
           Product: plasma

Attempt to use Plasma::ToolButton with for example QtGstremer ends up with
compilation error:

/usr/include/KDE/Plasma/../../plasma/widgets/toolbutton.h:189:9: error: ‘slots'
does not name a type

The offending header section:

    private slots:
        void setAnimationUpdate(qreal progress);
        qreal animationUpdate() const;

Seems to got broken in this commit:
https://projects.kde.org/projects/kde/kdelibs/repository/revisions/8d3d26776d7745c1d81b24099a1546add8f56309/


Manual replacing of slots by Q_SLOTS fixes the issue.
It seems that Plasma::ToolButton is the only affected widget.

Reproducible: Always

Steps to Reproduce:
1.  Create project that uses library that also uses keywords like slots etc.
(for example QtGstreamer) 
2. Add Plasma::ToolButton
3. Try to compile it
Actual Results:  
Compilation attempt ends with error.

Expected Results:  
Compilation should end with success.

-- 
You are receiving this mail because:
You are watching all bug changes.=


[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic