Skip to content
Permalink

Comparing changes

Choose two branches to see what’s changed or to start a new pull request. If you need to, you can also or learn more about diff comparisons.

Open a pull request

Create a new pull request by comparing changes across two branches. If you need to, you can also . Learn more about diff comparisons here.
base repository: nico-abram/blondie
Failed to load repositories. Confirm that selected base ref is valid, then try again.
Loading
base: main
Choose a base ref
...
head repository: hydro-project/blondie
Failed to load repositories. Confirm that selected head ref is valid, then try again.
Loading
compare: main
Choose a head ref
Able to merge. These branches can be automatically merged.
  • 3 commits
  • 12 files changed
  • 1 contributor

Commits on Aug 20, 2024

  1. Small cleanups and fixups

    - Moves bins into standard `src/bin` folder
    - Adds a basic test (TODO: doesn't work on gh-actions CI)
    - Reformat with new `rustfmt.toml`, rearrange some code, add some comments
    - Fix wrong error message for `GetVersionExA`
    - `cargo update`
    MingweiSamuel committed Aug 20, 2024
    Copy the full SHA
    e1baeb4 View commit details
  2. Copy the full SHA
    4e1223e View commit details

Commits on Aug 21, 2024

  1. multi-process proof-of-concept

    yay unsoundness:
    thread '<unnamed>' panicked at core\src\panicking.rs:221:5:
    unsafe precondition(s) violated: slice::get_unchecked requires that the index is within the slice
    note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace
    thread caused non-unwinding panic. aborting.
    error: test failed, to rerun pass `--test basic`
    
    Caused by:
      process didn't exit successfully: `D:\Projects\blondie\target\debug\deps\basic-10e35b968cd5dbed.exe` (exit code: 0xc0000409, STATUS_STACK_BUFFER_OVERRUN)
    MingweiSamuel committed Aug 21, 2024
    Copy the full SHA
    252b265 View commit details
Showing with 1,153 additions and 778 deletions.
  1. +3 −2 .github/workflows/blondie.yml
  2. +1 −18 .github/workflows/clippy.yml
  3. +639 −453 Cargo.lock
  4. +0 −2 Cargo.toml
  5. +10 −0 rustfmt.toml
  6. 0 src/{blondie_bin.rs → bin/blondie.rs}
  7. 0 src/{ → bin}/blondie_dtrace.rs
  8. +51 −0 src/error.rs
  9. +277 −303 src/lib.rs
  10. +147 −0 src/util.rs
  11. +9 −0 tests/basic.rs
  12. +16 −0 tests/multi.rs
5 changes: 3 additions & 2 deletions .github/workflows/blondie.yml
Original file line number Diff line number Diff line change
@@ -43,8 +43,9 @@ jobs:
- name: Build
run: cargo build --release --all-features --verbose

- name: Run tests
run: cargo test --release --all-features --verbose -- --nocapture
# TODO: Doesn't record any tracing events on github-actions
# - name: Run tests
# run: cargo test --release --all-features --verbose -- --nocapture

docs:
runs-on: windows-latest
19 changes: 1 addition & 18 deletions .github/workflows/clippy.yml
Original file line number Diff line number Diff line change
@@ -13,23 +13,6 @@ jobs:
- uses: actions/checkout@v2

- run: rustup component add clippy

# Workaround from https://github.com/actions-rs/clippy-check/issues/2#issuecomment-807878478
# - name: Check workflow permissions
# id: check_permissions
# uses: scherermichael-oss/action-has-permission@1.0.6
# with:
# required-permission: write
# env:
# GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
# - name: Run clippy action to produce annotations
# uses: actions-rs/clippy-check@v1
# if: steps.check_permissions.outputs.has-permission
# with:
# args: --all-targets -- -D warnings
# token: ${{ secrets.GITHUB_TOKEN }}
# - name: Run clippy manually without annotations
# if: ${{ !steps.check_permissions.outputs.has-permission }}
# run: cargo clippy --all-targets -- -D warnings

- name: Run clippy manually without annotations
run: cargo clippy --all-targets -- -D warnings
Loading