WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED INVALID
125247
Headers that use std::unique_ptr should #include <memory>
https://bugs.webkit.org/show_bug.cgi?id=125247
Summary
Headers that use std::unique_ptr should #include <memory>
Brian Burg
Reported
2013-12-04 13:04:27 PST
This needs to go into the style guide and style checker if it is something we want to encourage going forward.
Attachments
Add attachment
proposed patch, testcase, etc.
Darin Adler
Comment 1
2013-12-04 18:07:14 PST
I don’t agree that this belongs in the style guide. This follows the general rule that headers that use something need to include the file that something is defined in. I wouldn’t make a style rule for each thing. Why is it OK to use std::unique_ptr without including <memory>? Because <memory> is included in "config.h" perhaps?
László Langó
Comment 2
2013-12-04 23:16:56 PST
I agree with Darin. This doesn't belong in the style guide. But this just an opinion. :)
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug