https://bugs.kde.org/show_bug.cgi?id=358799
--- Comment #13 from Igor Kushnir ---
> Showing a warning is probably more difficult to get right (a nested event
> loop or code scattered across several functions).
Actually there *is* an easy way to show a notification in the shell embedded
area
https://bugs.kde.org/show_bug.cgi?id=358799
--- Comment #12 from Igor Kushnir ---
One more reason why the current behavior is reasonable: removing the default
parameter value breaks code that uses the function. Adding a new parameter with
a default value keeps the old code working as before. So
https://bugs.kde.org/show_bug.cgi?id=358799
Piotr Mierzwinski changed:
What|Removed |Added
Resolution|--- |FIXED
Status|REPORTED
https://bugs.kde.org/show_bug.cgi?id=358799
--- Comment #10 from Igor Kushnir ---
(In reply to Piotr Mierzwinski from comment #9)
> So the fix breaks different use case.
But why does it break a use case? Your proposed alternative of removing the
previous default parameter value can be a loss of
https://bugs.kde.org/show_bug.cgi?id=358799
--- Comment #9 from Piotr Mierzwinski ---
(In reply to Igor Kushnir from comment #8)
> The `= {} /* TODO */` part is intentional - implemented in
> https://commits.kde.org/kdevelop/ba2ecbaeba206644c1134569e6da40047096487a to
> fix another bug you
https://bugs.kde.org/show_bug.cgi?id=358799
Igor Kushnir changed:
What|Removed |Added
CC||igor...@gmail.com
--- Comment #8 from Igor
https://bugs.kde.org/show_bug.cgi?id=358799
Piotr Mierzwinski changed:
What|Removed |Added
Status|NEEDSINFO |REPORTED
Resolution|WAITINGFORINFO
https://bugs.kde.org/show_bug.cgi?id=358799
--- Comment #6 from Bug Janitor Service ---
Dear Bug Submitter,
This bug has been in NEEDSINFO status with no change for at least
15 days. Please provide the requested information as soon as
possible and set the bug status as REPORTED. Due to regular
https://bugs.kde.org/show_bug.cgi?id=358799
Justin Zobel changed:
What|Removed |Added
Status|REPORTED|NEEDSINFO
Resolution|---
https://bugs.kde.org/show_bug.cgi?id=358799
Piotr Mierzwinski changed:
What|Removed |Added
Version|git master |5.1.2
10 matches
Mail list logo