Hacktoberfest 2026: los issues que los mantenedores marcaron para octubre, abiertos y aptos para principiantes. Explorar issues de Hacktoberfest

Bug: ASCII PLY parsing fails on files with \r line endings due to binary mode

Abierto
#45 0 comentarios 0 reacciones 0 asignados Ver en GitHub

Nadie ha tomado este issue todavía.

Evaluación

Dificultad
2/5
Tiempo estimado
1-3 horas
Aptitud para principiantes
45/100
Tipo de issue
Error
Claridad
Bastante claro
Estado de actividad
Estancado
Stack tecnológico
cpp
Área
computer-graphics

Línea de trabajo

Start at PLYData(filename) and trace the ASCII parser used when the binary flag is false. Reproduce the failure with an ASCII PLY containing standalone \r blank lines, then verify that equivalent files using \n, \r\n, and \r parse successfully without treating blank lines as data.

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

Descripción

When loading an ASCII PLY file via PLYData(filename), the parser fails to properly handle empty lines containing standalone \r (Carriage Return) characters. This occurs because the file is opened in binary mode (preserving raw \r), but the ASCII parser does not explicitly skip them. As a result, lines with \r are treated as valid content, leading to crashes.

Expected behavior:
The parser should safely ignore empty lines regardless of line ending type (\n, \r\n, or \r).

Reproduction:

  • Create an ASCII PLY file with empty lines between header and data.
  • Load it with happly::PLYData plyIn("file.ply", false);
  • Parsing will fail when reading element data.

fily.ply:
ply
format ascii 1.0
element vertex 1
property float x
property float y
property float z
end_header

0.0 0.0 0.0

Lenguaje dominante
C++
Estrellas
406
Forks
74
Sin PR fusionados en 30 d

Guía de contribución

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

Primeros pasos

  1. Lee el issue completo y luego la guía de contribución del proyecto.
  2. Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
  3. Haz un fork del repositorio y trabaja en una rama.
  4. Abre un pull request que haga referencia al número del issue.

Más de nmwsharp/happly

Todos los issues de nmwsharp/happly

Issues similares

Más issues de C++

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.