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

Is there an expr directive #2149

Closed
tbm opened this issue Dec 22, 2022 · 1 comment
Closed

Is there an expr directive #2149

tbm opened this issue Dec 22, 2022 · 1 comment
Labels
documentation Related to the documentation

Comments

@tbm
Copy link
Contributor

tbm commented Dec 22, 2022

@jwiegley can you comment.

@simonmichael was wondering about the expr directive.

ledger3.text says:

@item expr
@findex expr
@c instance_t::expr_directive in textual.cc

There's no expr_directive in textual.cc. Does this actually exist?
@tbm tbm added the documentation Related to the documentation label Dec 22, 2022
@jwiegley
Copy link
Member

I guess it's not a separate function anymore? Here's the code that it represents:

    else if (keyword == "eval" || keyword == "expr") {
      // jww (2012-02-27): Make account into symbol scopes so that this
      // can be used to override definitions within the account.
      bind_scope_t bound_scope(*context.scope, *account);
      expr_t(b).calc(bound_scope);
    }

@tbm tbm closed this as completed in 0eac2c4 Dec 27, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
documentation Related to the documentation
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@jwiegley @tbm