fire-eggs / fire-eggs/YAGP

Mixed LF / CRLF causes confusion

Open
#45 0 comments 0 reactions 0 assignees View on GitHub
bug library
Dominant language
C#
Stars
8
Forks
3
PR merge metrics
No merged PRs in 30d

Description

I've now encountered a couple of GED files with mixed EOL. A good example is PDR.GED - the header has LF but the first record has CRLF.

The CRLF causes the parser to get confused and fails to recognise the record types. The output from the writer appends a NUL thanks to the CR.

see 01\PDR.ged

Contributor guide

No contributing guide indexed for this repository

Research direction

Start by reproducing the problem with 01\PDR.ged, comparing the LF header and CRLF first record described in the issue. Trace the parser and writer entry points to see how mixed EOL is handled; done means record types are recognized and the writer no longer appends a NUL for this input.

Written by the indexing model from the issue text.

Assessment

Tech stack
csharp
Domain
data
Issue type
Bug
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.