[
https://issues.apache.org/jira/browse/COR-41?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14319800#comment-14319800
]
jan iversen commented on COR-41:
--------------------------------
We do not need to tell which compilers are supported, we support what is
defined in our cmake files.
> Use of /* ... */ preferable to //... in STDC Code
> -------------------------------------------------
>
> Key: COR-41
> URL: https://issues.apache.org/jira/browse/COR-41
> Project: Corinthia
> Issue Type: Bug
> Affects Versions: 0.5
> Environment: All .c and .h files in the repository
> Reporter: Dennis E. Hamilton
> Priority: Trivial
> Labels: Apache-License-Headers, C
>
> The ASF Notices on .c and .h files use // ... single-line comment triggers
> instead of something like
> /* ...
> *
> ...
> */
> the only form that some compilers recognize as STDC comments. The use of
> -Wall and STDC compile options will produce warnings in some products, such
> as most versions of Visual Studio, clogging the error log with warning
> messages.
> PROPOSAL:
> 1. Agree that it is OK to change these to /* ... */ when the particular
> files are touched for some purpose, such as clearing warnings out of a strict
> STDC compile. Enough compilers warn about this, apparently for down-level
> STDC compatibility.
> (In the VC++ standard libraries, there is a file deep in the Windows SDK
> headers that has a single //... that annoys me enough to actually fix it in
> the library. I'd like to fix this with some sort of platform quirk patch but
> I haven't dug enough into the include-path rules to get that working.
> Someday soon ... .)
> 2. Fix these on an opportunistic basis. There is no due date but it would
> be nice that all new ones be in the universally-STDC form. (Headers and code
> files that *require* C++ are another story and they will have different file
> extensions as well.)
> 3. Pray that rat, the IP checker, recognizes these notices either way.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
