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.

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

Replace DontExecLink by something better? #992

Open
linas opened this issue Nov 10, 2016 · 4 comments
Open

Replace DontExecLink by something better? #992

linas opened this issue Nov 10, 2016 · 4 comments

Comments

@linas
Copy link
Member

linas commented Nov 10, 2016

Perhaps it would be best to retire DontExecLink and replace all uses with LocalQuoteLink ?

See #704

@ngeiswei
Copy link
Member

ngeiswei commented Nov 11, 2016

I think it make sense. Will take care of that once I'm done supporting LocalQuoteLink.

@ngeiswei
Copy link
Member

Actually I don't think DontExecLink can be replaced by a
LocalQuoteLink. For instance I don't see how to prevent the result
from being executed, in the following example

(BindLink
   (TypedVariableLink
      (VariableNode "$A")
      (TypeNode "BindLink")
   )
   (EvaluationLink
      (PredicateNode "URE:BC:and-BIT")
      (VariableNode "$A")
   )
   (DontExec (Variable "$A")))

If (DontExecLink (Variable "$A")) is replaced by (LocalQuoteLink (Variable "$A")) then $A never gets replaced by its matching
BindLink.

@ngeiswei
Copy link
Member

It seems DontExecLink could be replaced by LocalQuoteLink on links, but not on variable nodes. So it still necessary.

@linas
Copy link
Member Author

linas commented Jul 12, 2017

OK, that actually sounds right. I think that perhaps additional insight and ideas can be gotten from https://www.gnu.org/software/guile/manual/html_node/Syntax-Case.html

I'd like to keep this open as a possible to-do enhancement, and so will change the name of the issue to reflect this.

@linas linas changed the title Replace DontExecLink by LocalQuoteLink ? Replace DontExecLink by something better? Jul 12, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants