Election-Tech-Initiative / Election-Tech-Initiative/electionguard-core2

✨ UTF-8 Simplification

Open
#169 0 comments 0 reactions 0 assignees View on GitHub
enhancement
Dominant language
C#
Stars
14
Forks
12
PR merge metrics
No merged PRs in 30d

Description

### Is there an existing issue for this?

- [X] I have searched the existing issues

### Suggestion

Because of other serialization changes, the changes to serialize.hpp might be able to be simpler.

static string toJson(const electionguard::Manifest &serializable)

In line 591, the regex might not be needed because of changes that were made on the c# side.

return regex_replace(manifestStr, regex("\\\\\\\\u"), "\\u");

### Possible Implementation

_No response_

### Anything else?

_No response_

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.