[ 
https://issues.apache.org/jira/browse/GEODE-2741?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16077978#comment-16077978
 ] 

ASF GitHub Bot commented on GEODE-2741:
---------------------------------------

Github user vjr commented on a diff in the pull request:

    https://github.com/apache/geode-native/pull/102#discussion_r126127440
  
    --- Diff: src/CMakeLists.txt ---
    @@ -17,6 +17,15 @@ project(nativeclient)
     
     list(APPEND CMAKE_MODULE_PATH ${CMAKE_SOURCE_DIR}/../cmake)
     
    +if(CMAKE_GENERATOR MATCHES Win64*)
    +  if ((CMAKE_GENERATOR MATCHES "Visual Studio") AND 
(CMAKE_GENERATOR_TOOLSET STREQUAL ""))
    +    message(WARNING "GEODE expects that a user must provode -Thost=x64 if 
you are using a"
    --- End diff --
    
    Typo - "provode" should be "provide" ?


> Use C++11 shared pointer over custom shared pointer
> ---------------------------------------------------
>
>                 Key: GEODE-2741
>                 URL: https://issues.apache.org/jira/browse/GEODE-2741
>             Project: Geode
>          Issue Type: Improvement
>          Components: native client
>            Reporter: Addison
>            Assignee: Jacob S. Barrett
>
> *Context*
> Now that the Native Client is compatible with C++11, we can use its shared 
> pointer over the custom shared pointer we use today.
> *Definition of Done*
> The custom shared pointer is nowhere to be found in the codebase



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

Reply via email to