Created attachment 501257 [details] [diff] [review] patch 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)
Status: NEW → RESOLVED
Last Resolved: 7 years ago
No longer depends on: 610267
Resolution: --- → FIXED
Target Milestone: --- → mozilla2.2
You need to log in before you can comment on or make changes to this bug.