-
Notifications
You must be signed in to change notification settings - Fork 154
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Use of angle brackets around file names for include statements #22
Comments
Not so sure about this one. Is this really necessary? |
I suggest to reconsider the consequences of the following wording from the section "16.2 Source file inclusion" in the standard specification for the programming language "C++".
|
Markus,
After reading https://gcc.gnu.org/onlinedocs/cpp/Include-Syntax.html, I
think "" is correct. It allows xed-interface.h to find the other header
files in the current directory, even if they are not in the system include
area or user does not use -I<path-to-xed>. It also prevents issues where
xed-interface.h is in a private directory, but incorrectly pulls in the
rest of the header files from a system dirctory.
Robert
…On Fri, Dec 23, 2016 at 1:34 AM, Markus Elfring ***@***.***> wrote:
I suggest to reconsider the consequences of the following wording
<http://c0x.coding-guidelines.com/6.10.2.html> from the section "16.2
Source file inclusion"
<http://open-std.org/jtc1/sc22/wg21/docs/papers/2013/n3797.pdf> in the
standard specification for the programming language "C++".
…
The named source file is searched for in an implementation-defined manner. If this search is not supported, or if the search fails, the directive is reprocessed as if it read
#include <h-char-sequence> new-line
…
- How do you think about to avoid "a duplicated file search" here?
- Would you like to restrict the searched directories
<https://gcc.gnu.org/onlinedocs/cpp/Include-Syntax.html> for header
files of your software?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
<#22 (comment)>, or mute
the thread
<https://github.com/notifications/unsubscribe-auth/AB2NWr5ea5Nlq08AkHQgpsc2-AdbU3ghks5rK2t3gaJpZM4LUXKN>
.
|
There are different opinions about the handling of the involved implementation-defined behaviour.
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Would you like to replace any double quotes by angle brackets around file names for include statements?
The text was updated successfully, but these errors were encountered: