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

Wasm bindings for Parside #4

Open
wants to merge 31 commits into
base: main
Choose a base branch
from
Open

Wasm bindings for Parside #4

wants to merge 31 commits into from

Conversation

Artemkaaas
Copy link
Contributor

The wasm crate adding this PR is based on parside implementation done in: #2

Artemkaaas and others added 23 commits February 8, 2023 17:50
@Artemkaaas
Option with defining groups
d75d2c5
@Artemkaaas
Added stubs for missing groups + code refactoring
31cf6d5
@Artemkaaas
Added group constructors and example of to_string/to_bytes
72a54c1
@Artemkaaas
Simplified group definition
36b5b1c
@Artemkaaas
Refactored parser util
02f70a6
@Artemkaaas
Added uniffi bindings
0c9caaf
@Artemkaaas
Added stubs for primitives parsers
95a364e
@Artemkaaas
Refactored groups definition
a72e123
@Artemkaaas
Updated parside to use the latest cesride
69eaddf
@Artemkaaas
Updated cesride version
91f692b
@Artemkaaas
Exclude Uniffi in favor of third crate
c80bad4
@Artemkaaas
Removed exporting of cesride primitives
f77d21e
@Artemkaaas
Merge remote-tracking branch 'origin/main' into initial-implementation
9788a84 
# Conflicts:
#	Cargo.toml
#	src/lib.rs
@DmitryKuzmenko
Update for moved *_size in cesride
f5796f6
@DmitryKuzmenko
Cargo fmt
5831ff8
@DmitryKuzmenko
Clippy fixes
716d53c
@DmitryKuzmenko
Update for new full_size functions. Fix-ups for b64/b2
0d8753f
@DmitryKuzmenko
Handle empty stream in nom group parser
9983e44
@DmitryKuzmenko
Support anyhow error
eb0c3df
@DmitryKuzmenko
Formatting
f3c6c59
@DmitryKuzmenko
Bump cesride version
96cdd30
@DmitryKuzmenko
Implemented full_size for groups
43cf74d
@DmitryKuzmenko
Use memcpy copying in qb*
72cb1d8
@Artemkaaas Artemkaaas requested a review from m00sey as a code owner April 17, 2023 09:17
@m00sey
Copy link
Member

m00sey commented Aug 3, 2023

@m00sey to try merge/fix tests

@m00sey m00sey self-assigned this Aug 3, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@m00sey m00sey Awaiting requested review from m00sey m00sey is a code owner

Assignees

@m00sey m00sey

Labels
pending close
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@Artemkaaas @m00sey @pfeairheller @DmitryKuzmenko