This pull request just exists to have a place to put comments.  Please don't pull!

The "|| 1" is indeed redundant. I think that snuck in from an earlier experiment I ran. Removing it. getline() should return false if we're suddenly unable to read from stdin.