Zulip Chat Archive
Stream: PR reviews
Topic: Quick style PRs
Winston Yin (尹維晨) (Dec 03 2023 at 01:43):
Two easy-to-review PRs for style:
#8776 removes some namespace prefixes such asEdit: I went overboard!Set.,Function.,Filter,LocalHomeomorphwhen the namespace is alreadyopened. E.g.Set.univtouniv.- #8780 reorganises the variables in
SmoothManifoldWithCornersto remove clutter.
Also #8702 (WIP) removes a bunch of explicit @foo _ _ _ _ in favour of Lean4's named arguments foo (x := y). I'd like to know what people think about this. I think it improves readability, reduces brittleness of code when argument order changes, and sets an example of the style of Lean4.
Yaël Dillies (Dec 03 2023 at 07:50):
Reviewed #8702
Last updated: May 02 2025 at 03:31 UTC