[C++] Remove an unnecessary null pointer check
I maintainer di solito rispondono entro 1 giorno
Valutazione
- Difficoltà
- 1/5
- Tempo stimato
- Meno di un'ora
- Idoneità per principianti
- 78/100
Direzione di ricerca
La modifica riguarda il distruttore di ArrowLog in cpp/src/arrow/util/logging.cc, intorno alle righe 258-262, dove un controllo di null protegge un delete. Leggi quel distruttore e verifica che il puntatore venga eliminato solo lì, poi rimuovi il controllo ridondante, dato che delete su un puntatore nullo non ha effetto. Completato quando la protezione è rimossa, il file continua a compilare e la build e i test C++ esistenti passano.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
Describe the enhancement requested
An extra null pointer check is not needed in the destructor for the class “ArrowLog”.
Component(s)
C++
- Lingua principale
- C++
- Stelle
- 17.2k
- Fork
- 4.4k
- Merge medio
- 3g 9h
- PR unite (30g)
- 69
Preparare l'ambiente
- Include un Dockerfile o un file Docker Compose
- Ha un modello di pull request
- Leggi la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di apache/arrow
-
[R][Parquet] ParquetReaderProperties does not expose page_checksum_verification, so a corrupt page CRC reads silentlyForse di nuovo libera Una pull request per questa issue è stata chiusa senza essere unita. ApertaComponent: R good-first-issue Type: enhancement
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
apache/arrow#53138 · 2 commenti ·
I maintainer di solito rispondono entro 1 giorno
-
[C++][Python] IPC reader rejects a DictionaryEncoding without indexType, which the format allowsForse già presa Una pull request collegata a questa issue è aperta o già unita. ApertaComponent: C++ Component: Python Type: bug
Difficoltà 2/5 1-3 ore Idoneità per principianti 68/100
I maintainer di solito rispondono entro 1 giorno
-
[R] Silent loss of metadata integrity for date-time and numeric attributes in write_parquet()/read_parquet()Forse di nuovo libera @james-finn-travers l’ha presa 13 giorni fa e non c’è nessuna pull request aperta. ApertaComponent: R Type: bug
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
apache/arrow#51695 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
-
[C++][Parquet] Plaintext-footer files written with AES_GCM_CTR_V1 record AES_GCM_V1 as the encryption algorithm and cannot be readForse di nuovo libera Una pull request per questa issue è stata chiusa senza essere unita. ApertaType: bug
Difficoltà 2/5 1-3 ore Idoneità per principianti 88/100
I maintainer di solito rispondono entro 1 giorno
-
[R] Expose ignore_extra_columns and pad_short_rows CSV parse optionsForse già presa Una pull request collegata a questa issue è aperta o già unita. ApertaComponent: R good-first-issue Type: enhancement
Difficoltà 2/5 1-3 ore Idoneità per principianti 76/100
apache/arrow#51659 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
Tutte le issue di apache/arrow
Issue simili
-
binding:web core docs p3
Difficoltà 2/5 1-3 ore Idoneità per principianti 85/100
RunanywhereAI/runanywhere-sdks#1123 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 88/100
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 82/100
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 66/100
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 62/100
NVIDIA-RTX/NRI#277 ·
I maintainer di solito rispondono entro 1 giorno