-
Notifications
You must be signed in to change notification settings - Fork 3.5k
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
GH-36950: [C++] Change std::vector<std::shared_ptr<Field>> to use it'…
…s alias: FieldVector (#37101) ### Rationale for this change Clarity and Maintainability ### What changes are included in this PR? 1. Changed all occurrence of std::vector<std::shared_ptr<Field>> in **type_fwd.h**、**type.h** and **type.cc** to FieldVector for consistency. 2. Use move to eliminate vector copy in sparse and dense union's constructor ### Are these changes tested? Covered by existing tests ### Are there any user-facing changes? No * Closes: #36950 Authored-by: jsjtxietian <jsjtxietian@outlook.com> Signed-off-by: Antoine Pitrou <antoine@python.org>
- Loading branch information
1 parent
757e0d5
commit 9cabd94
Showing
3 changed files
with
57 additions
and
74 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters