jediwhale/fitsharp — pull request #105 by kimgr (2012-08-14 11:17:51+00:00)
Title: Parse operators trim whitespace

After our discussion on Symbol trimming, I've done some work to make it more consistent.

Cell/CellBase/Parse/CellSubstring now have a Content property containing the Text as shown in the browser. This currently implies only that it's trimmed.

CellSubstring just returns its current Text; I left a TODO there, because I'm not sure if this is correct behavior.

The following operators have been updated to use Content instead of Text:
- ParseBlank
- ParseBoolean
- ParseByteArray
- ParseDate
- ParseNull
- ParseNullArray
- ParseQuotedString
- ParseSymbol
- CheckSymbolSave

I'm sure there are more operators that should use Content over Text, but I didn't want to change them without understanding the ramifications; we only use parse operators.

I also tried to improve unit test coverage of parse operators at large and added a tidy HTML page with symbol operators in it, to check that this works end-to-end

Happy to hear what you think about this!

- Kim