[
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 <[email protected]>
Date: 2017-04-03T15:09:35Z
IGNITE-3582: Removed IgnireError passing by pointer
commit 1ea13faca27efd9e56ed337ad0bfb109a775924d
Author: Igor Sapego <[email protected]>
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)
