bramp / bramp/ffmpeg-cli-wrapper

Make probe result objects more immutable

Abierto
#384 5 comentarios 0 reacciones 0 asignados Ver en GitHub
enhancement
Lenguaje dominante
Java
Estrellas
1.9k
Forks
424
Merge medio
5 h 35 min
PR fusionados (30 d)
2

Descripción

Probe result objects (`FFmpegProbeResult`, `FFmpegStream`, `FFmpegFormat`, etc.) are currently mutable — fields are public and collections (e.g., streams list, tags map) are modifiable after construction.

We should make these objects more immutable to prevent accidental modification and improve thread safety.

This is part of the API modernization discussed in #316.

**Scope:**
- Return unmodifiable collections from getter methods (e.g., `Collections.unmodifiableList()` for streams)
- Consider making fields final where possible
- Ensure Gson deserialization still works (may need custom deserializers or builder pattern)

Depends on: #382, #383
Related: #316

Guía de contribución

No hay ninguna guía de contribución indexada para este repositorio

Línea de trabajo

Comienza localizando las clases de resultados de la sonda, incluidas FFmpegProbeResult, FFmpegStream y FFmpegFormat, e inspecciona sus campos públicos y getters de colecciones. Comprueba la ruta de deserialización de Gson y el trabajo de las dependencias #382 y #383. Se considera completado cuando los objetos de resultado impiden la mutación de las colecciones y utilizan campos final cuando sea posible, sin romper la deserialización de Gson.

Escrito por el modelo de indexación a partir del texto del issue.

Evaluación

Stack tecnológico
java
Área
tooling
Tipo de issue
Refactorización
Dificultad
4/5
Tiempo estimado
3-5 días
Estado de actividad
Tranquilo
Claridad
Bastante claro
Aptitud para principiantes
45/100

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.