Skip to content

WIP: Implement configuration objects#186

Open
FSMaxB wants to merge 38 commits intodevelopfrom
configuration
Open

WIP: Implement configuration objects#186
FSMaxB wants to merge 38 commits intodevelopfrom
configuration

Conversation

@FSMaxB
Copy link
Collaborator

@FSMaxB FSMaxB commented Jul 8, 2017

This is a work in progress of implementing configuration as discussed in #177

@FSMaxB FSMaxB force-pushed the configuration branch 2 times, most recently from 5614b5c to 44228ce Compare July 15, 2017 08:21
@FSMaxB FSMaxB mentioned this pull request Jan 29, 2018
Closed
@FSMaxB
Copy link
Collaborator Author

FSMaxB commented Feb 1, 2018 •
edited
Loading

  • Allocators with userdata
  • Creating configurations
  • Changing allocators and userdata in configuration
  • Making use of the configuration options in the code via global default configuration
  • Parsing with configuration
  • Printing with configuration
  • Other functions with configuration?
  • More tests
  • Documentation

@FSMaxB FSMaxB force-pushed the configuration branch 8 times, most recently from 7d60183 to 55de863 Compare February 3, 2018 18:15
@FSMaxB FSMaxB mentioned this pull request Feb 13, 2018
Closed
@FSMaxB FSMaxB mentioned this pull request Feb 25, 2018
Open
@FSMaxB FSMaxB added this to the 1.8 milestone Feb 25, 2018
@FSMaxB FSMaxB mentioned this pull request Feb 28, 2018
Closed
FSMaxB added 9 commits March 25, 2018 13:00
FSMaxB added 25 commits March 25, 2018 23:31
@FSMaxB
Gitignore: add CLion files
3ebc061
@FSMaxB
Rename internal_hooks -> internal_configuration, cJSON_New_item -> cr…
677f0cb 
…eate_item
@FSMaxB
Put format into internal_configuration
27977ad
@FSMaxB
Put buffer_size into internal_configuration
7030dc7
@FSMaxB
cJSON_Delete: Extract delete_item with internal_configuration
d4e81cf
@FSMaxB
cJSON_ParseWithOpts: Extract pasrse with internal_configuration
f02f79e 
Also introduces a allow_data_after_json property in the internal
configuration.
@FSMaxB
internal_configuration: Add case_sensitive
ba8fe0f
@FSMaxB
default_configuration: Macro for the internal_configuration defaults
515d11f
@FSMaxB
cJSON_Compare: Extract compare with internal_configuration
dd1ba72
@FSMaxB
Add cJSON_Allocators new style allocator struct
98e0b58
@FSMaxB
allocation helpers for allocating with a configuration
877fac0
@FSMaxB
cJSON_CreateConfiguration, cJSON_ConfigurationChange{Allocators,Userd…
9d801d6 
…ata}
@FSMaxB
cJSON_ConfigurationChangeParseEnd
88c39fa 
Add a pointer to an end position of parsing to the cJSON_Configuration
object. (Essentially like return_parse_end, but as offset instead of
pointer).
@FSMaxB
cJSON_ConfigurationChangePrebufferSize
1a8f732
@FSMaxB
cJSON_ConfigurationChangeFormat
78b5bed
@FSMaxB
cJSON_ConfigurationChangeCaseSensitivity
eeaaaac
@FSMaxB
cJSON_ConfigurationChangeAllowDataAfterJson
ae9dc3e
@FSMaxB
cJSON_CreateConfig: Don't allow configuration, always use default
691a83a
@FSMaxB
cJSON_ConfigurationChangeParseEnd -> cJSON_ConfigurationGetParseEnd
d2d1912 
This is probably a better approach than potentially having a pointer
that points to garbage on the stack and gets written to by cJSON.
@FSMaxB
cJSON_DuplicateConfiguration
050f982
@FSMaxB
Change name from Configuration to Context
064eec8
@FSMaxB
Context: Add duplicate_recursive for cJSON_Duplicate
e8f56bd
@FSMaxB
cJSON_MakeDuplicateRecursive
409c2aa
@FSMaxB
parse: Pull length calculation out.
1b001ab
@FSMaxB
Reorder the context fields
0873867
@FSMaxB FSMaxB mentioned this pull request Mar 28, 2018
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

1.8

Development

Successfully merging this pull request may close these issues.

1 participant

@FSMaxB