Skip to content
This repository was archived by the owner on Mar 20, 2025. It is now read-only.

Use QXmlStreamReader internally #620

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

lnjX
Copy link
Member

@lnjX lnjX commented Apr 28, 2024

  • StreamOpen: Impl fromXml() using QXmlStreamReader
  • XmppSocket: Parse whole stream through QXmlStreamReader

PR check list:

  • Document your code
  • Add \since QXmpp 1.X, QXMPP_EXPORT
  • Update doc/doap.xml
  • Add unit tests
  • Format the code: Run clang-format -i src/<edited-file(s)> tests/<edited-file(s)>

@lnjX lnjX force-pushed the xml-stream-reader branch from f813cac to 5f28f52 Compare May 18, 2024 22:15
@lnjX lnjX force-pushed the xml-stream-reader branch 2 times, most recently from 2a7fcad to aefedd8 Compare August 22, 2024 18:03
@lnjX lnjX force-pushed the xml-stream-reader branch from aefedd8 to 805873a Compare November 8, 2024 16:43
@lnjX lnjX force-pushed the xml-stream-reader branch from 805873a to 63fd483 Compare March 6, 2025 21:38
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant