fix: correct arg type of prctl::set_timerslack #2505

SteveLauC · 2024-09-15T10:02:18Z

What does this PR do

The ns argument of function nix::sys::prctl::set_timerslack() should have type c_ulong, before this PR, we are using an u64, which causes issue #2497.

This PR changes the type to c_ulong.

Closes #2497

Checklist:

I have read CONTRIBUTING.md
I have written necessary tests and rustdoc comments
A change log has been added if this PR modifies nix's API

SteveLauC · 2024-09-15T10:24:19Z

Well, that Cirrus CI issue happens again...

Using built-in Git...
Cloning refs/heads/gh-readonly-queue/master/pr-2505-7d75bbc9d6cfe91b903c91f39ed8b500ec65b8d1...
Failed to clone: couldn't find remote ref "refs/heads/gh-readonly-queue/master/pr-2505-7d75bbc9d6cfe91b903c91f39ed8b500ec65b8d1"!

SteveLauC added 2 commits September 15, 2024 18:00

fix: correct arg type of prctl::set_timerslack

42997b3

test: update test

31f0883

SteveLauC added this pull request to the merge queue Sep 15, 2024

github-merge-queue bot removed this pull request from the merge queue due to failed status checks Sep 15, 2024

SteveLauC merged commit b6f9cc3 into nix-rust:master Sep 15, 2024
38 checks passed

SteveLauC deleted the fix/incorrect_PR_SET_TIMERSLACK_arg_type branch September 15, 2024 10:24

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

fix: correct arg type of prctl::set_timerslack #2505

fix: correct arg type of prctl::set_timerslack #2505

SteveLauC commented Sep 15, 2024

SteveLauC commented Sep 15, 2024 •

edited

Loading

fix: correct arg type of prctl::set_timerslack #2505

fix: correct arg type of prctl::set_timerslack #2505

Conversation

SteveLauC commented Sep 15, 2024

What does this PR do

Checklist:

SteveLauC commented Sep 15, 2024 • edited Loading

SteveLauC commented Sep 15, 2024 •

edited

Loading