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

[Piotr Mierzwinski]

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

--- Comment #9 from Piotr Mierzwinski <piotr.mierzwin...@gmail.com> ---
(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 reported :). Locations of the feature and the test in
> the current code:
> $ git grep -n '/\* TODO \*/'
> plugins/clang/codegen/adaptsignatureassistant.cpp:213:               
> newSignature.defaultParams[i] = QStringLiteral("{} /* TODO */");
> plugins/clang/tests/test_assistants.cpp:469:        << "class Foo { void
> bar(bool b, int i = 0, char c = {} /* TODO */); };"

So the fix breaks different use case.

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