Closed Bug 623132 Opened 10 years ago Closed 10 years ago
Add constructor for ns
Case Insensitive CString Comparator (which inherits from ns CString Comparator) to placate CLang
nsCaseInsensitiveCStringComparator is missing a user defined constructor, but in mozStorageStatement.cpp a const variable of this type is defined. This is not valid c++. For more information see "Default initialization of const variable of a class type requires user-defined default constructor" in http://clang.llvm.org/compatibility.html#c++
The reporter's summary and initial comment were both lame. I'm merely adjusting the summary and providing a better link. I am not passing judgement on the quality of the bug report. http://clang.llvm.org/compatibility.html#default_init_const
Summary: Missing constructor → Add constructor for nsCaseInsensitiveCStringComparator (which inherits from nsCStringComparator) to placate CLang
Comment on attachment 501257 [details] [diff] [review] patch I have the same question as in bug 623116 comment 3, but anyway bsmedberg should review in XPCOM.
Attachment #501257 - Flags: review?(smontagu) → review?(benjamin)
You need to log in before you can comment on or make changes to this bug.