You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@celix.apache.org by ErjanAltena <gi...@git.apache.org> on 2017/01/31 18:49:41 UTC

[GitHub] celix pull request #19: Adding extern "C" to headers in case __cplusplus is ...

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 <er...@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.
---

[GitHub] celix pull request #19: Adding extern "C" to headers in case __cplusplus is ...

Posted by asfgit <gi...@git.apache.org>.
Github user asfgit closed the pull request at:

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


---
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.
---