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

Change the boolean markers from T/F to 1/0 #36

Closed
ghost opened this issue Sep 16, 2013 · 2 comments
Closed

Change the boolean markers from T/F to 1/0 #36

ghost opened this issue Sep 16, 2013 · 2 comments
Labels

Comments

@ghost
Copy link

ghost commented Sep 16, 2013

Got this idea from reading logzero's post here:
#27 (comment)

This is a modification he made was to replace T/F with 1/0.

The motivation for this would be to free up "T" to be used as the TYPE marker in the optimized container format instead of '$' and free up "F" and "f" to be used for the float32/64 types instead of "D" and "d".

Obviously would be a fairly big breaking change.

@kxepal
Copy link
Collaborator

kxepal commented Sep 16, 2013

+1

Have nothing to argue. F marker more sensible to floating values since double is just 64-bit float. And I can recall some cases when we had needed in T marker, but it was already used.

@ghost
Copy link
Author

ghost commented Jan 15, 2014

Not fixing; no massive win for making this fundamentally breaking change. Also makes the markers inconsistent with the friendly-ASCII design.

@ghost ghost closed this as completed Jan 15, 2014
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant