Skip to content

Commit

Permalink
Docs: fix typos and grammar (#51547)
Browse files Browse the repository at this point in the history 
doc: Corrected some sentences.

---------

Co-authored-by: Christian Guinard <[email protected]>
Co-authored-by: Fredrik Ekre <[email protected]>
  • Loading branch information
@masterujjval @christiangnrd @fredrikekre
3 people committed Oct 8, 2023
1 parent 341e9d0 commit e81c8e3
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions doc/src/devdocs/valgrind.md
View file
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@ assumption works fine in most instances but fails miserably for a just-in-time c
`julia`. For this reason it is crucial to pass `--smc-check=all-non-file` to `valgrind`, else
code may crash or behave unexpectedly (often in subtle ways).

In some cases, to better detect memory errors using Valgrind it can help to compile `julia` with
In some cases, to better detect memory errors using Valgrind, it can help to compile `julia` with
memory pools disabled. The compile-time flag `MEMDEBUG` disables memory pools in Julia, and
`MEMDEBUG2` disables memory pools in FemtoLisp. To build `julia` with both flags, add the following
line to `Make.user`:
Expand All @@ -20,7 +20,7 @@ line to `Make.user`:
CFLAGS = -DMEMDEBUG -DMEMDEBUG2
```

Another thing to note: if your program uses multiple workers processes, it is likely that you
Another thing to note: if your program uses multiple worker processes, it is likely that you
want all such worker processes to run under Valgrind, not just the parent process. To do this,
pass `--trace-children=yes` to `valgrind`.

Expand Down Expand Up @@ -57,7 +57,7 @@ to `valgrind` as well.

## Additional spurious warnings

This section covers Valgrind warnings which cannot be added to the
This section covers Valgrind warnings that cannot be added to the
suppressions file yet are nonetheless safe to ignore.

### Unhandled rr system calls
Expand Down

2 comments on commit e81c8e3

@nanosoldier
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Executing the daily package evaluation, I will reply here when finished:

@nanosoldier runtests(isdaily = true)

@nanosoldier
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The package evaluation job you requested has completed - possible new issues were detected.
The full report is available.

Please sign in to comment.