37
89%
master: 86%

Ran 03 May 2019 07:20PM UTC

Jobs 1

Files 1

Run time 47min

Badge

Embed ▾

pending completion

Build # 37

Build Type

Pull #10

travis-ci

Committed by

web-flow

Commit Message

TryFrom refactoring & misc updates

During review of #7, several possible improvements were revealed.
First of all, now that
[TryFrom](https://doc.rust-lang.org/std/convert/trait.TryFrom.html) is
stable, the auxiliary `TryIntoProcessHandle` trait is no longer
needed.

This change
* Refactors the library to use `TryFrom` trait for `Pid ->
  ProcessHandle` and `Child -> ProcessHandle` conversions.

  `TryFrom` trait implementation must reference a type defined in this
  crate, so either `ProcessHandler` or `Pid` had to be converted from
  a type alias to a 1 field tuple.

  It seems like users interact with `ProcessHandle` less often than
  with `Pid`, so converting `ProcessHandle` won't sacrifice ergonomics
  much.

  Example (retrieving the process handle):

  Before:
  ```rust
    let handle = pid.try_into_process_handle()?;
  ```

  After:
  ```rust
    let handle: ProcessHandle = pid.try_into()?;
  ```

* Adds `--verbose` flags to Cirrus CI scripts

* Adds Cirrus CI badge to readme.

* Reconfigures Appveyor CI to use rustup directly, since the old
  script downloaded v1.16 instead of current stable (v1.34)

Pull Request Pull Request #10: TryFrom refactoring & misc updates

Run Details

66 of 73 relevant lines covered (90.41%)

0.9 hits per line

Jobs

ID	Job ID	Ran	Files	Coverage
1	37.1	03 May 2019 07:20PM UTC	0	90.41	Travis Job 37.1

luser / read-process-memory / 37
89%
master: 86%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 37

luser / read-process-memory / 37 89% master: 86%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 37

luser / read-process-memory / 37
89%
master: 86%

README BADGES
x