drewnoakes/metadata-extractor

PngMetadataReader - investigate if addDirectory call is redundant

Open

#414 aperta il 18 lug 2019

Vedi su GitHub
 (0 commenti) (0 reazioni) (0 assegnatari)Java (470 fork)batch import
buggood-first-issuehelp wanted

Metriche repository

Star
 (2411 star)
Metriche merge PR
 (Nessuna PR mergiata in 30 g)

Descrizione

Is the addDirectory on line 180 redundant? It happens further down also outside of the catch.

https://github.com/drewnoakes/metadata-extractor/blob/81143f746e6774eb7625e374b3f6e56396288d1a/Source/com/drew/imaging/png/PngMetadataReader.java#L168-L185

Guida contributor

Direzione di ricerca
Esamina la riga 180 di PngMetadataReader.java. Determina se la chiamata addDirectory all'interno del blocco catch è ridondante dato che successivamente si verifica un'altra chiamata addDirectory. Rivedi il flusso del codice ed esegui i test esistenti dopo aver rimosso la riga per verificare che non ci siano regressioni.
Tech stack
java
Dominio
data
Tipo issue
Refactoring
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Meno di 1 ora
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Datata
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
Basic JavaFamiliarity with metadata extractor
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
80