Some updates #15
Open
Some updates #15
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
renamed file properly to disk_partitions
added documentation for the DiskPartition type
added a link to the newly created DiskParition documentation
added DiskPartition type documentation
existsFile is deprecated, and now using fileExists instead
replaced ..colon with 0..colon, because ..b is deprecated to 0..b.
replaced ..10 with 0..10, because ..b is deprecated to 0..b.
cuchar is deprecated, thus replaced with char
implicit conversion from string to cstring, thus added the cast to avoid CStringConv compile errors in the future
type pragmas follow the type name; this form of writing pragmas is deprecated
See corresponding Defect; DivByZeroError is deprecated, thus use DivByZeroDefect instead
added the pid_kill function, which will send a SIGKILL to the specified PID.
added pid_kill procedure for killing the specified pid. This opens the process and then terminates it. Raises an OSError for an error
Do to casting errors, i've added casts to where I was getting them.
i've added some checks for OpenProcess return variable.
added the pid_kill proc to the psutil.nim file, also exporting the platform so user can access all the function without needing to do "psutil/platform_linux" or "psutil/platform_windows"
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
I've added a pid_kill procedure to kill the specified pid, and some updates to the psutil library, such as casting where needed, and adding check for OpenProcess. Also removed deprecated feature in favor of the supported ones.