Skip to content

Commit

Permalink
Fix typos in @static docstring (#54227)
Browse files Browse the repository at this point in the history
Followup for premature merge of #54206

Implements @giordano's suggestions

cc @Sbozzolo
  • Loading branch information
LilithHafner committed Apr 25, 2024
1 parent 3d6da2c commit 8f6418e
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions base/osutils.jl
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@
Partially evaluate an expression at macro expansion time.
This is useful in cases where a construct would be invalid in some cases, such as a `ccall`
to a os-dependent function, or macros defined in packages that are not imported.
to an os-dependent function, or macros defined in packages that are not imported.
`@static` requires a conditional. The conditional can be in an `if` statement, a ternary
operator, or `&&`\`||`. The conditional is evaluated by recursively expanding macros,
Expand All @@ -16,9 +16,9 @@ macro-expanded (and lowered or executed).
# Example
Suppose we want to parse an expression `expr` that is valid only on MacOS. We could solve
Suppose we want to parse an expression `expr` that is valid only on macOS. We could solve
this problem using `@static` with `@static if Sys.isapple() expr end`. In case we had
`expr_apple` for MacOS and `expr_others` for the other operating systems, the solution with
`expr_apple` for macOS and `expr_others` for the other operating systems, the solution with
`@static` would be `@static Sys.isapple() ? expr_apple : expr_others`.
"""
macro static(ex)
Expand Down

2 comments on commit 8f6418e

@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.