gvsafronov/rapidjson
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Clarify documentation on RemoveMember, EraseMember ordering

Browse Source
This commit is contained in:
Sean Leather 2014-10-27 15:04:26 +02:00
parent 8fc0e57926
commit 716907b9fb
1 changed files with 10 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

15
include/rapidjson/document.h
View File

@ -1035,7 +1035,9 @@ public:
//! Remove a member in object by its name. //! Remove a member in object by its name.
/*! \param name Name of member to be removed. /*! \param name Name of member to be removed.
\return Whether the member existed. \return Whether the member existed.
\note Removing member is implemented by moving the last member. So the ordering of members is changed. \note This function may reorder the object members. Use \ref
EraseMember(ConstMemberIterator) if you need to preserve the
relative order of the remaining members.
\note Linear time complexity. \note Linear time complexity.
*/ */
bool RemoveMember(const Ch* name) { bool RemoveMember(const Ch* name) {
@ -1057,8 +1059,9 @@ public:
//! Remove a member in object by iterator. //! Remove a member in object by iterator.
/*! \param m member iterator (obtained by FindMember() or MemberBegin()). /*! \param m member iterator (obtained by FindMember() or MemberBegin()).
\return the new iterator after removal. \return the new iterator after removal.
\note Removing member is implemented by moving the last member. So the ordering of members is changed. \note This function may reorder the object members. Use \ref
\note Use \ref EraseMember(ConstMemberIterator) instead, if you need to rely on a stable member ordering. EraseMember(ConstMemberIterator) if you need to preserve the
relative order of the remaining members.
\note Constant time complexity. \note Constant time complexity.
*/ */
MemberIterator RemoveMember(MemberIterator m) { MemberIterator RemoveMember(MemberIterator m) {
@ -1085,7 +1088,8 @@ public:
\pre IsObject() == true && \ref MemberBegin() <= \c pos < \ref MemberEnd() \pre IsObject() == true && \ref MemberBegin() <= \c pos < \ref MemberEnd()
\return Iterator following the removed element. \return Iterator following the removed element.
If the iterator \c pos refers to the last element, the \ref MemberEnd() iterator is returned. If the iterator \c pos refers to the last element, the \ref MemberEnd() iterator is returned.
\note Other than \ref RemoveMember(MemberIterator), this function preserves the ordering of the members. \note This function preserves the relative order of the remaining object
members. If you do not need this, use the more efficient \ref RemoveMember(MemberIterator).
\note Linear time complexity. \note Linear time complexity.
*/ */
MemberIterator EraseMember(ConstMemberIterator pos) { MemberIterator EraseMember(ConstMemberIterator pos) {
@ -1097,7 +1101,8 @@ public:
\param last iterator following the last member to remove \param last iterator following the last member to remove
\pre IsObject() == true && \ref MemberBegin() <= \c first <= \c last <= \ref MemberEnd() \pre IsObject() == true && \ref MemberBegin() <= \c first <= \c last <= \ref MemberEnd()
\return Iterator following the last removed element. \return Iterator following the last removed element.
\note Other than \ref RemoveMember(MemberIterator), this function preserves the ordering of the members. \note This function preserves the relative order of the remaining object
members.
\note Linear time complexity. \note Linear time complexity.
*/ */
MemberIterator EraseMember(ConstMemberIterator first, ConstMemberIterator last) { MemberIterator EraseMember(ConstMemberIterator first, ConstMemberIterator last) {