[C++] Remove an unnecessary null pointer check
メンテナーはふだん 1 日以内に返信
評価
調査の方向性
変更対象は cpp/src/arrow/util/logging.cc の258〜262行目付近にある ArrowLog のデストラクタで、null チェックが delete を囲んでいます。そのデストラクタを読み、ポインタがそこでしか delete されていないことを確認してください。そのうえで、null ポインタへの delete は何もしないため、冗長なチェックを削除します。ガードが削除され、ファイルがコンパイルでき、既存の C++ ビルドとテストが通れば完了です。
索引モデルが issue の本文から書いたものです。
説明
Describe the enhancement requested
An extra null pointer check is not needed in the destructor for the class “ArrowLog”.
Component(s)
C++
- 主要言語
- C++
- スター
- 17.2k
- フォーク
- 4.4k
- 平均マージ
- 3日 9時間
- マージ済み PR（30日）
- 69
環境構築
- Dockerfile または Docker Compose ファイルあり
- プルリクエストのテンプレートあり
- コントリビューションガイドを読む
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
apache/arrow のほかの issue
-
[R][Parquet] ParquetReaderProperties does not expose page_checksum_verification, so a corrupt page CRC reads silently再び着手できるかも このイシューのプルリクエストはマージされずにクローズされました。 オープンComponent: R good-first-issue Type: enhancement
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
apache/arrow#53138 · コメント 2 件 ·
メンテナーはふだん 1 日以内に返信
-
[C++][Python] IPC reader rejects a DictionaryEncoding without indexType, which the format allows対応中かも このイシューにリンクされたプルリクエストがオープン中、またはマージ済みです。 オープンComponent: C++ Component: Python Type: bug
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
メンテナーはふだん 1 日以内に返信
-
[R] Silent loss of metadata integrity for date-time and numeric attributes in write_parquet()/read_parquet()再び着手できるかも @james-finn-travers が 14 日前に担当しましたが、オープン中のプルリクエストはありません。 オープンComponent: R Type: bug
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
apache/arrow#51695 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
-
[C++][Parquet] Plaintext-footer files written with AES_GCM_CTR_V1 record AES_GCM_V1 as the encryption algorithm and cannot be read再び着手できるかも このイシューのプルリクエストはマージされずにクローズされました。 オープンType: bug
難易度 2/5 1〜3時間 初心者へのやさしさ 88/100
メンテナーはふだん 1 日以内に返信
-
[R] Expose ignore_extra_columns and pad_short_rows CSV parse options対応中かも このイシューにリンクされたプルリクエストがオープン中、またはマージ済みです。 オープンComponent: R good-first-issue Type: enhancement
難易度 2/5 1〜3時間 初心者へのやさしさ 76/100
apache/arrow#51659 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
似ている issue
-
backlog bug
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
EchoTools/nevr-runtime#653 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
InfiniTimeOrg/InfiniTime#2492 ·
-
accessibility good first issue hacktoberfest help wanted
難易度 2/5 1〜3時間 初心者へのやさしさ 74/100
OpenChemistry/avogadrolibs#3184 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
Tracktion/tracktion_engine#475 ·
メンテナーはふだん 1 日以内に返信