[CXX-377] clang 3.5 warning, dbclientcursor.h Created: 20/Oct/14 Updated: 03/Apr/23 Resolved: 09/Dec/14 |
|
| Status: | Closed |
| Project: | C++ Driver |
| Component/s: | Build |
| Affects Version/s: | legacy-0.0-26compat-2.6.5 |
| Fix Version/s: | legacy-0.0-26compat-2.6.6 |
| Type: | Bug | Priority: | Major - P3 |
| Reporter: | Allan Bazinet | Assignee: | Unassigned |
| Resolution: | Done | Votes: | 0 |
| Labels: | legacy-cxx | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Issue Links: |
|
||||||||
| Description |
|
Using the clang 3.5 compiler, the following warning is issued during compilation of the 26compat branch. While this may not be of immediate concern, it seems odd enough to report. clang++ -O3 -g -std=c+11 -stdlib=libc+ -I ../libs/ami/include -emit-llvm -c ^~~ |
| Comments |
| Comment by Andrew Morrow (Inactive) [ 09/Dec/14 ] |
|
https://github.com/mongodb/mongo-cxx-driver/commit/a3fb94cb14f95a92b9c33fa04e2bfb51d3b8c1d1 |
| Comment by Githook User [ 09/Dec/14 ] |
|
Author: {u'username': u'acmorrow', u'name': u'Andrew Morrow', u'email': u'acm@mongodb.com'}Message: Cherry pick of server commit db59c175c736609970b52f91e30fc2277e22b25b |
| Comment by Andrew Morrow (Inactive) [ 21/Oct/14 ] |
|
I think I agree that it would be good for the 26compat headers to be able to be consumed without warnings by clang-3.5. I'm putting this into the 'planning' bucket so we don't lose track of it. |
| Comment by Allan Bazinet [ 20/Oct/14 ] |
|
Andrew, Thanks for the update; we'll apply the fixes in Given that the documentation for the legacy branch still discourages production use, I would encourage backport of this fix to the 26compat branch. |
| Comment by Andrew Morrow (Inactive) [ 20/Oct/14 ] |
|
Hi - Thanks for the report. This has actually already been fixed in the server master branch, and that fix has been backported to the C++ driver legacy branch. The fix will be available in the upcoming legacy-1.0.0-rc1 release expected later this week. See Please note though that we are unlikely to fix this in the 26compat branch unless the fix for |