[kdevelop] [Bug 358799] Usage of "Update definition signature" in "Adapt Signature" breaks declaration of function if its argument has default value and without it

2022-12-04 Thread Igor Kushnir
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

[kdevelop] [Bug 358799] Usage of "Update definition signature" in "Adapt Signature" breaks declaration of function if its argument has default value and without it

2022-12-02 Thread Igor Kushnir
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

[kdevelop] [Bug 358799] Usage of "Update definition signature" in "Adapt Signature" breaks declaration of function if its argument has default value and without it

2022-12-01 Thread Piotr Mierzwinski
https://bugs.kde.org/show_bug.cgi?id=358799 Piotr Mierzwinski changed: What|Removed |Added Resolution|--- |FIXED Status|REPORTED

[kdevelop] [Bug 358799] Usage of "Update definition signature" in "Adapt Signature" breaks declaration of function if its argument has default value and without it

2022-12-01 Thread Igor Kushnir
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

[kdevelop] [Bug 358799] Usage of "Update definition signature" in "Adapt Signature" breaks declaration of function if its argument has default value and without it

2022-12-01 Thread Piotr Mierzwinski
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

[kdevelop] [Bug 358799] Usage of "Update definition signature" in "Adapt Signature" breaks declaration of function if its argument has default value and without it

2022-11-30 Thread Igor Kushnir
https://bugs.kde.org/show_bug.cgi?id=358799 Igor Kushnir changed: What|Removed |Added CC||igor...@gmail.com --- Comment #8 from Igor

[kdevelop] [Bug 358799] Usage of "Update definition signature" in "Adapt Signature" breaks declaration of function if its argument has default value and without it

2022-11-30 Thread Piotr Mierzwinski
https://bugs.kde.org/show_bug.cgi?id=358799 Piotr Mierzwinski changed: What|Removed |Added Status|NEEDSINFO |REPORTED Resolution|WAITINGFORINFO

[kdevelop] [Bug 358799] Usage of "Update definition signature" in "Adapt Signature" breaks declaration of function if its argument has default value and without it

2022-11-24 Thread Bug Janitor Service
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

[kdevelop] [Bug 358799] Usage of "Update definition signature" in "Adapt Signature" breaks declaration of function if its argument has default value and without it

2022-11-10 Thread Justin Zobel
https://bugs.kde.org/show_bug.cgi?id=358799 Justin Zobel changed: What|Removed |Added Status|REPORTED|NEEDSINFO Resolution|---

[kdevelop] [Bug 358799] Usage of "Update definition signature" in "Adapt Signature" breaks declaration of function if its argument has default value and without it

2017-10-03 Thread Piotr Mierzwinski
https://bugs.kde.org/show_bug.cgi?id=358799 Piotr Mierzwinski changed: What|Removed |Added Version|git master |5.1.2