Skip to content

CPP: Deprecate Expr.getKind() and Stmt.getKind().#1519

Merged
pavgust merged 1 commit intogithub:masterfrom
geoffw0:depkind
Jun 27, 2019
Merged

CPP: Deprecate Expr.getKind() and Stmt.getKind().#1519
pavgust merged 1 commit intogithub:masterfrom
geoffw0:depkind

Conversation

@geoffw0
Copy link
Contributor

@geoffw0 geoffw0 commented Jun 27, 2019

These predicates expose details of our dbscheme that no user should have to care about. I don't think these are used in our libraries or queries any more, and I hope none of our users have gravitated towards using them.

Discussion welcome.

@geoffw0 geoffw0 added the C++ label Jun 27, 2019
@geoffw0 geoffw0 requested a review from a team as a code owner June 27, 2019 17:34
Copy link
Contributor

@jbj jbj left a comment

Choose a reason for hiding this comment

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

Agreed. If a query needs to depend on low-level details of the dbscheme, it can also use the stmts or exprs relations directly.

@pavgust, is there any historical or field use of these predicates we should be aware of?

@pavgust
Copy link
Contributor

pavgust commented Jun 27, 2019

I know they've been used to write generic structural equality libraries, but that use case is now served by GVN or HashCons. I think it's appropriate to start the deprecation cycle.

@pavgust pavgust merged commit da7591d into github:master Jun 27, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

Comments