You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
The official rust style guidelines state that redundant prefixes should be avoided. I consider the LispCons part of these type names redundant. As the style guide says, module names should be used to disambiguate - I consider the lists module name perfectly fit for this purpose - or renaming imports in cases where that's necessary.
The current type names are long and take up frustratingly large amounts of line real-estate, for no benefit.
The text was updated successfully, but these errors were encountered:
I would have liked to have done this more like C++ or other type systems and defined EndChecks and CircularChecks inside of LispCons and had LispCons::EndChecks.
The lists module is itself not sufficient in my opinion as a name space for these. We could move some of it to a cons.rs which would better map.
...and
LispConsCircularChecks
The official rust style guidelines state that redundant prefixes should be avoided. I consider the
LispCons
part of these type names redundant. As the style guide says, module names should be used to disambiguate - I consider thelists
module name perfectly fit for this purpose - or renaming imports in cases where that's necessary.The current type names are long and take up frustratingly large amounts of line real-estate, for no benefit.
The text was updated successfully, but these errors were encountered: