celix-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From ErjanAltena <...@git.apache.org>
Subject [GitHub] celix pull request #19: Adding extern "C" to headers in case __cplusplus is ...
Date Tue, 31 Jan 2017 18:49:41 GMT
GitHub user ErjanAltena opened a pull request:

    https://github.com/apache/celix/pull/19

    Adding extern "C"  to headers in case __cplusplus is defined

    Added the precompiler directive stated below to all public headers.
    
    Developers using celix in an C++ environment don't have to to put these directives around
the inclusion of the headers any more if this pull request is merged.
    
    I tested it with some examples.
    ```
    
    #ifdef __cplusplus
    extern "C" {
    #endif   
    <<original code>>
    #ifdef __cplusplus
    }
    #endif   
    
    
    ```

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/ErjanAltena/celix cpp_headers

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/celix/pull/19.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #19
    
----
commit ce0586bcb34f35a47bdf961a20de630ae31b82da
Author: Erjan Altena <erjanaltena@gmail.com>
Date:   2017-01-31T18:15:33Z

    Adding extern "C"  to headers in case __cplusplus is defined

----


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

Mime
View raw message