Skip to content

Commit

Permalink
nitpick: crazy -> overboard (#30068)
Browse files Browse the repository at this point in the history 
[ci skip]
Following some discussion on slack, some consensus grew around removing unhelpful and ableist usage of "crazy".
  • Loading branch information
@miguelraz @ViralBShah
miguelraz authored and ViralBShah committed Nov 18, 2018
1 parent 8c5baf3 commit d32b30e
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion doc/src/manual/performance-tips.md
View file
Original file line number Diff line number Diff line change
Expand Up @@ -707,7 +707,7 @@ type-domain.

## The dangers of abusing multiple dispatch (aka, more on types with values-as-parameters)

Once one learns to appreciate multiple dispatch, there's an understandable tendency to go crazy
Once one learns to appreciate multiple dispatch, there's an understandable tendency to go overboard
and try to use it for everything. For example, you might imagine using it to store information,
e.g.

Expand Down

2 comments on commit d32b30e

@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 benchmark build, I will reply here when finished:

@nanosoldier runbenchmarks(ALL, 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.

Your benchmark job has completed - possible performance regressions were detected. A full report can be found here. cc @ararslan

Please sign in to comment.