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

Ascending function detected #1132

Closed
ccecvb opened this issue Jun 28, 2024 · 2 comments
Closed

Ascending function detected #1132

ccecvb opened this issue Jun 28, 2024 · 2 comments
Assignees
@gquerret
Labels
CABL rule idea

Comments

@ccecvb
Copy link

ccecvb commented Jun 28, 2024

This came up during our 12.2 to 12.8 migration

Before 12.3 OpenEdge wrongfully interpreted ascending(x) as asc(x), see https://community.progress.com/s/article/Compiler-interprets-ASCENDING-as-ASC-function.

We found code where the ASC function had been expanded incorrectly to ASCENDING.
OpenEdge 12.2 did not have any issues with that.

Maybe this could be part of a "code migration rule set", it could help fixing this kind of issue before the actual migration.
@gquerret gquerret self-assigned this Jun 28, 2024
@gquerret gquerret added the CABL label Jun 28, 2024 — with Linear
@gquerret Linear
Copy link
Contributor

Yes, easy to implement, and makes sense to have issues reported in this case.

@gquerret gquerret added the rule idea label Jun 28, 2024 — with Linear
@gquerret gquerret changed the title rule idea: ascending function detected Ascending function detected Jun 28, 2024
@gquerret Linear
Copy link
Contributor

gquerret commented Jul 3, 2024

Merged in develop branch

@gquerret gquerret closed this as completed Jul 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@gquerret gquerret

Labels
CABL rule idea
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@gquerret @ccecvb