-
Type: Task
-
Resolution: Done
-
Priority: Major - P3
-
Affects Version/s: None
-
Component/s: Aggregation Framework
-
None
-
Fully Compatible
-
Query 2017-01-23
The convenience of the Value(const char* value) constructor doesn't seem to outweigh the potential risk (see SERVER-23356 and SERVER-25991) for erroneously attempting to construct a string literal with an embedded null byte.
The Value(StringData value) and user-defined literal constructor for StringData from SERVER-24651 seem sufficient for syntactic convenience.
- is related to
-
SERVER-23356 ExpressionSubstrCPTest uses non-NUL terminated char*, and does not properly construct strings with embedded null bytes
- Closed
-
SERVER-25991 WithNullCharacter* test cases in ExpressionSubstrCPTest should use StringData literal
- Closed