Closed Bug 1154188 Opened 10 years ago Closed 10 years ago

security/certverifier/NSSCertDBTrustDomain.cpp:824:3: error: 'UniquePtr' was not declared in this scope (non-SPS)

Categories

(Core :: Security: PSM, defect)

All
FreeBSD
defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla40
Tracking Status
firefox40 --- fixed

People

(Reporter: jbeich, Assigned: jbeich)

References

Details

Attachments

(1 file)

SPS has the following bootlegging thanks to bug 1151829 security/certverifier/Unified_cpp_certverifier0.cpp:20: security/certverifier/NSSCertDBTrustDomain.cpp:15: security/certverifier/../manager/ssl/src/nsNSSCertificate.h:14: security/certverifier/../manager/ssl/src/nsNSSShutDown.h:11: dist/include/mozilla/CondVar.h:16: dist/include/GeckoProfiler.h:204: dist/include/GeckoProfilerImpl.h:16: dist/include/mozilla/UniquePtr.h on platforms lacking SPS profiler (Linux/PPC, BSDs, Solaris) it leads to In file included from security/certverifier/Unified_cpp_certverifier0.cpp:20: security/certverifier/NSSCertDBTrustDomain.cpp:824:3: error: use of undeclared identifier 'UniquePtr' UniquePtr<char, void(&)(char*)> ^ security/certverifier/NSSCertDBTrustDomain.cpp:824:17: error: expected '(' for function-style cast or type construction UniquePtr<char, void(&)(char*)> ~~~~^ security/certverifier/NSSCertDBTrustDomain.cpp:824:25: error: expected expression UniquePtr<char, void(&)(char*)> ^ security/certverifier/NSSCertDBTrustDomain.cpp:824:31: error: expected '(' for function-style cast or type construction UniquePtr<char, void(&)(char*)> ~~~~^ security/certverifier/NSSCertDBTrustDomain.cpp:824:32: error: expected expression UniquePtr<char, void(&)(char*)> ^ security/certverifier/NSSCertDBTrustDomain.cpp:825:5: error: use of undeclared identifier 'fullLibraryPath' fullLibraryPath(PR_GetLibraryName(dir, "nssckbi"), PR_FreeLibraryName); ^ security/certverifier/NSSCertDBTrustDomain.cpp:826:8: error: use of undeclared identifier 'fullLibraryPath' if (!fullLibraryPath) { ^ ...
Attached patch explicit #include (deleted) — Splinter Review
Attachment #8592125 - Flags: review?(brian)
FFS... failures are just piling on top of each other. Can ppl stop messing up with those damn headers ?
Comment on attachment 8592125 [details] [diff] [review] explicit #include Review of attachment 8592125 [details] [diff] [review]: ----------------------------------------------------------------- ::: security/certverifier/NSSCertDBTrustDomain.cpp @@ +12,3 @@ > #include "ExtendedValidation.h" > #include "OCSPRequestor.h" > #include "certdb.h" Please put the include of mozilla/UniquePtr.h here (alphabetical order)
Attachment #8592125 - Flags: review?(brian) → review+
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla40
Assignee: nobody → jbeich
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: