Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

mlrAnatDBPut-fix #40

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

mlrAnatDBPut-fix #40

wants to merge 1 commit into from

Conversation

steevelaquitaine
Copy link
Contributor

Start writing old entries from line 2 of subjectPageScan because line 1 entries are empty. This causes the code to crash when creating a new SID.txt file.

Start writing old entries from line 2 of subjectPageScan because line 1
entries are empty. This causes the code to crash when creating a new
SID.txt file.
@dbirman
Copy link
Collaborator

dbirman commented Apr 29, 2016

Is this to fix the bug on this page: http://gru.stanford.edu/doku.php/mlranatdb/index where there are blank lines? It seems like this fix just ignores the blanks, but if no blanks were on the page it would fail to re-write the first line. I think the right fix is to figure out why it posts blank lines at all, not sure where that happens.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants