CarbonManagerV2: Fix enabling carbons

qxmpp-project / qxmpp

Cross-platform C++ XMPP client and server library

401 stars 197 forks source link

CarbonManagerV2: Fix enabling carbons #545

Closed melvo closed 1 year ago

melvo commented 1 year ago

PR check list:

[x] Document your code
[x] Add \since QXmpp 1.X, QXMPP_EXPORT
[x] Fix doxygen warnings (see log when building with -DBUILD_DOCUMENTATION=ON)
[x] Update doc/doap.xml
[x] Add unit tests
[x] Format the code: Run clang-format -i src/<edited-file(s)> tests/<edited-file(s)>

melvo commented 1 year ago

@lnj The regular way for enabling carbons would be to not add a to attribute but I could not find a way to omit that. When I do not set to, it is automatically set to the server's JID.