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

Always return struct by pointer #315

Open

saracen wants to merge 1 commit into risor-io:main from saracen:struct-pointer

saracen commented Feb 11, 2025

Currently, if you use:

type A struct {
  B string
}

type Config struct {
  A A
}

var config Config
risor.NewConfig(risor.WithGlobal("config", &config))

You'll get type error: cannot set field B for the Risor script:

config.A.B = "hello"
config

This PR ensures that for structs we return by pointer allowing us to set the fields of a nested struct.


          Always return struct by pointer

7db75be

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet