Skip to content

MongoDB::Array interface improvements#3013

Closed
matejk wants to merge 78 commits intopocoproject:masterfrom
matejk:mongodb-array-member-access-improvements
Closed

MongoDB::Array interface improvements#3013
matejk wants to merge 78 commits intopocoproject:masterfrom
matejk:mongodb-array-member-access-improvements

Conversation

@matejk
Copy link
Copy Markdown
Contributor

@matejk matejk commented May 27, 2020

Two improvements:

  • Use std::size_t instead of int in get function to be consistent with type returned from size()
  • Array-specific add functions that automatically set stringified sequence number for element name to simplify adding members to array
  • added test function for MongoDB array

obiltschnig and others added 20 commits February 17, 2020 09:00
…ingPattern

Write only Filename to the Log Output
…s and callback implementation to verify the response if the server returns any response
…orClientConn

Configuration to receive OCSP stapling response for client connection…
* Fixed pocoproject#2945

* Added unit tests for pocoproject#2945

* Dissalow iterator on empty Var (pocoproject#2945)

* Updated unit tests for pocoproject#2945

* More concise unit tests for pocoproject#2945

* Removed some more clutter (pocoproject#2945)
…::prepare() must prepare with underlying type, not Poco::Data::Keywords::null
Zip and SevenZip do not depend on Util, XML, JSON
Fix set padding call for new versions of OpenSSL
@obiltschnig obiltschnig self-assigned this May 29, 2020
@obiltschnig obiltschnig added this to the Release 1.11.0 milestone May 29, 2020
@obiltschnig
Copy link
Copy Markdown
Member

Thanks, could you please re-target the PR against the devel branch?

@matejk matejk force-pushed the mongodb-array-member-access-improvements branch from 4469ed6 to 399511f Compare May 30, 2020 11:15
…w helper functions to add elements to an array.
@matejk matejk force-pushed the mongodb-array-member-access-improvements branch from 399511f to 06617f7 Compare May 30, 2020 11:20
@matejk
Copy link
Copy Markdown
Contributor Author

matejk commented May 30, 2020

OK, I will create new PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

9 participants