8000 refactor(web): iron_init/setup logic by CBenoit · Pull Request #763 · Devolutions/IronRDP · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

refactor(web): iron_init/setup logic #763

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.

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

Merged
merged 1 commit into from
Apr 21, 2025

Conversation

CBenoit
Copy link
Member
@CBenoit CBenoit commented Apr 21, 2025
  • iron_init is renamed to setup, so that

    • init is the function for initializing the WASM module.
    • setup is the function taking parameters for logger and performing other setting up operations.
    • Since setup may be called after init is called, it’s possible to use some functions of the WASM module before calling setup if necessity arises.
  • Common initializion code is moved to an hidden "internal" module of iron-remote-desktop crate, that is thus not part of the public API.

  • Restored the "IronRDP is ready" debug log when setup is called.

- `iron_init` is renamed to `setup`, so that
  - `init` is the function for initializing the WASM module.
  - `setup` is the function taking parameters for logger and performing
    other setting up operations.
  - Since `setup` may be called after `init` is called, it’s possible to
    use some functions of the WASM module before calling `setup` if
    necessity arises.

- Common initializion code is moved to an hidden "internal" module of
  iron-remote-desktop crate, that is thus not part of the public API.

- Restored the "IronRDP is ready" debug log when `setup` is called.
@CBenoit CBenoit requested a review from RRRadicalEdward April 21, 2025 08:42
@CBenoit CBenoit enabled auto-merge (squash) April 21, 2025 08:42
Copy link
Collaborator
@RRRadicalEdward RRRadicalEdward left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Clean 🧹

@CBenoit CBenoit merged commit 45884c5 into master Apr 21, 2025
10 checks passed
@CBenoit CBenoit deleted the refactor/iron-remote-desktop-setup-function branch April 21, 2025 12:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants
0