ignite-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "ASF GitHub Bot (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (IGNITE-3582) CPP: Replace pointers with references in function signatures where it's possible.
Date Mon, 03 Apr 2017 16:26:41 GMT

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

ASF GitHub Bot commented on IGNITE-3582:
----------------------------------------

GitHub user isapego opened a pull request:

    https://github.com/apache/ignite/pull/1724

    IGNITE-3582 CPP: Replaced pointers with references in function signatures where it's possible

    

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

    $ git pull https://github.com/gridgain/apache-ignite ignite-3582

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

    https://github.com/apache/ignite/pull/1724.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 #1724
    
----
commit d729b459c63110b36cf395e858905701e6820875
Author: Igor Sapego <isapego@gridgain.com>
Date:   2017-04-03T15:09:35Z

    IGNITE-3582: Removed IgnireError passing by pointer

commit 1ea13faca27efd9e56ed337ad0bfb109a775924d
Author: Igor Sapego <isapego@gridgain.com>
Date:   2017-04-03T15:31:36Z

    IGNITE-3582: Refactored BinaryMapReader::GetNext

----


> CPP: Replace pointers with references in function signatures where it's possible.
> ---------------------------------------------------------------------------------
>
>                 Key: IGNITE-3582
>                 URL: https://issues.apache.org/jira/browse/IGNITE-3582
>             Project: Ignite
>          Issue Type: Task
>          Components: platforms
>    Affects Versions: 1.6
>            Reporter: Igor Sapego
>            Assignee: Igor Sapego
>              Labels: cpp
>             Fix For: 2.0
>
>
> Replace passing arguments by pointers with passing by references where it's possible.
Places to look at:
> {{BinaryMapReader::GetNext}}.
> The following argument: {{IgniteError* err}}.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

Mime
View raw message