Revamp code related to "pretty" JSON serialization

[chisholm]

No description provided.

[codecov-io]

Codecov Report

Merging #192 into master will decrease coverage by 0.04%.
The diff coverage is 96.42%.

@@            Coverage Diff             @@
##           master     #192      +/-   ##
==========================================
- Coverage   99.54%   99.49%   -0.05%     
==========================================
  Files          62       62              
  Lines        6538     6520      -18     
==========================================
- Hits         6508     6487      -21     
- Misses         30       33       +3

Impacted Files	Coverage Δ
stix2/base.py	99.47% <100%> (-0.01%)	⬇️
stix2/test/test_utils.py	100% <100%> (ø)	⬆️
stix2/utils.py	98.2% <96.15%> (-1.8%)	⬇️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update ee059b0...f09960d. Read the comment docs.

[gtback]

This looks good to me. It seems more elegant (fewer levels of nesting). I'm a bit concerned by the drop in coverage, but as long as the tests are still passing, I'm pretty sure the effective behavior hasn't changed.

[gtback]

@emmanvg can you look at this and provide any feedback, too?

[emmanvg]

I am ok with the approach. There shouldn't be any more comparison error being raised. Though, when a property is not found the object will simply be printed out of order.