patchmanager icon indicating copy to clipboard operation
patchmanager copied to clipboard

[Doc] Remove/Improve build-breaking "Dummy" method docs

Open nephros opened this issue 1 year ago • 1 comments

Turns out these methods are not so dummy after all. (See #452)

One is a Signal, which shouldn't have an implementation at all.

The other is indeed dead code, but lack of usage o its parameter broke the build. Instad of wrapping it in Q_UNUSED, lets just not have an implementation, just the qdoc comment.

This now does silence qdoc, and does not break the build.

nephros avatar May 28 '24 21:05 nephros

Will look at this, try to comprehend the storyline which developed over years and approve, soon™.

Olf0 avatar May 29 '24 03:05 Olf0

Hey @Olf0 if you have the time it would be nice if you could approve and merge this.

The issue currently prevents building master on OBS.

nephros avatar Nov 06 '24 11:11 nephros