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
slightly simplify and clean up the qmail-send job helper functions #213
base: main
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is good. By returning the job number in job_avail() and using that in job_open() one entire for(;;) loop has been eliminated.
3de0c01
to
9bb3571
Compare
pass_selprep needs adjustment for the new return value of job_avail ( Line 1004 in 9bb3571
|
My "Legacy Open Source Fridays" ensemble-programming session will be submitting a PR shortly (probably this Friday) to add tests around the existing behaviors of |
I would welcome that. Since I plan to make them |
9bb3571
to
c6aa444
Compare
c6aa444
to
ed143a9
Compare
ed143a9
to
9604513
Compare
job_avail() already knew which was the next available job slot. job_open() would scan for the same number again. In case there would be no job free by then it returned -1, which was never checked, but used as an array index.
9604513
to
d831e7d
Compare
No description provided.