Julia: Rename Base.Order.Forward to lowercase

Created on 28 Mar 2020  路  3Comments  路  Source: JuliaLang/julia

Since Forward and Reverse are singleton instances like nothing and missing, it should probbaly have been named forwardordering and reverseordering instead for consistency with the Julia style guide and convention.

This is clearly breaking, but might be worth considering for 2.0.

Most helpful comment

Same for MergeSort, QuickSort, and InsertionSort.

All 3 comments

If we follow the guideline, the instances of RoundingMode (e.g. RoundNearest) should also be lowercase.
I do not strongly oppose changing them to lowercase. However, the changes reveal that they are not types, but make it difficult to distinguish them from ordinary variables or functions. I鈥檓 not sure that is quite right. :confused:

Same for MergeSort, QuickSort, and InsertionSort.

I'm concerned with the usability (especially for people who are not expert English readers) of introducing more names without delimited word boundaries in place of names with word boundaries.

Was this page helpful?
0 / 5 - 0 ratings

Related issues

StefanKarpinski picture StefanKarpinski  路  216Comments

quinnj picture quinnj  路  179Comments

StefanKarpinski picture StefanKarpinski  路  113Comments

StefanKarpinski picture StefanKarpinski  路  141Comments

StefanKarpinski picture StefanKarpinski  路  131Comments