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

remove pretty spaces when using pretty tabs #443

Closed
wants to merge 1 commit into from
Closed

remove pretty spaces when using pretty tabs #443

wants to merge 1 commit into from

Conversation

qaisjp
Copy link

@qaisjp qaisjp commented Aug 20, 2018

@hawicz
Copy link
Member

hawicz commented Aug 21, 2018

Yeah... no, but thanks anyway. If we're going to do anything here, it should be validation of the flags in one place, at the start of json_object_to_json_string_length(), and returning an error if incompatible ones are detected.

@hawicz hawicz closed this Aug 21, 2018
@qaisjp
Copy link
Author

qaisjp commented Aug 21, 2018

¯\_(ツ)_/¯ okay

@qaisjp qaisjp deleted the patch-1 branch August 21, 2018 03:15
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.

None yet

2 participants