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.

Sign up for GitHub

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

Jump to bottom

Performance improvents to LiteralSerializer #265

Merged
merged 1 commit into from
Oct 20, 2024
Merged

Performance improvents to LiteralSerializer #265

merged 1 commit into from
Oct 20, 2024
+95 −108

Conversation

brantburnett
Copy link
Contributor

  • No longer boxes when serializing some value types such as int, long, and Guid
  • Don't unnecessarily load CultureInfo.InvariantCulture for Guid
  • Use "." separator for PathSegmentStyle.Label regardless of the value of "explode" (this correctly follows the specification)

**Breaking changes

  • LiteralSerializer and HeaderSerializer are now static classes

@brantburnett
Performance improvents to LiteralSerializer
6485dbb 
- No longer boxes when serializing some value types such as int, long,
  and Guid
- Don't unnecessarily load CultureInfo.InvariantCulture for Guid
- Use "." separator for PathSegmentStyle.Label regardless of the value
  of "explode" (this correctly follows the specification)

**Breaking changes
- LiteralSerializer and HeaderSerializer are now static classes
@brantburnett
Copy link
Contributor Author

/merge

@centeredgebot centeredgebot bot merged commit cb04589 into main Oct 20, 2024
11 checks passed
@centeredgebot centeredgebot bot deleted the path-serializer branch October 20, 2024 15:59
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
hacktoberfest-accepted
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@brantburnett