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

Replace macro functions with real functions #2171

Closed
389-ds-bot opened this issue Sep 13, 2020 · 3 comments
Closed

Replace macro functions with real functions #2171

389-ds-bot opened this issue Sep 13, 2020 · 3 comments
Labels
performance Issue impacts performance
Milestone
FUTURE

Comments

@389-ds-bot
Copy link

Cloned from Pagure issue: https://pagure.io/389-ds-base/issue/49112

We have a number of places that use macros instead of functions. We should replace this with real function calls as this is a poor practice that is hard to debug.
@389-ds-bot 389-ds-bot added the performance Issue impacts performance label Sep 13, 2020
@389-ds-bot 389-ds-bot added this to the FUTURE milestone Sep 13, 2020
@389-ds-bot
Copy link
Author

Comment from firstyear (@Firstyear) at 2017-02-11 23:04:31

Metadata Update from @Firstyear:

  • Issue assigned to Firstyear
  • Issue set to the milestone: FUTURE

@389-ds-bot
Copy link
Author

Comment from firstyear (@Firstyear) at 2017-03-10 01:47:11

Metadata Update from @Firstyear:

  • Issue close_status updated to: None
  • Issue tagged with: Performance

@vashirov
Copy link
Member

We will do the change gradually, not all at the same time. Closing this as wontfix.

@vashirov vashirov closed this as not planned Won't fix, can't repro, duplicate, stale Jul 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
performance Issue impacts performance
Projects
None yet
Milestone
FUTURE
Development

No branches or pull requests
2 participants
@vashirov @389-ds-bot