-
Notifications
You must be signed in to change notification settings - Fork 242
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[ DRY ] Streamline Function.*
hierarchy
#2570
Comments
Can you give us a preview of what you think this would entail? |
Well, the obvious 'minimal' fix/streamlining I envisaged after #2568 would be to:
But, the more I worked on that PR, the more threads seemed to get pulled... so it's a bit hard to see exactly how things might cash out? Eg.
Maybe it isn't too dramatic as it first seemed to me, cf. |
@JacquesCarette The other way to answer your question might be: I have no idea, my brain hurts, so I'd like someone else to think about the possible knock-on consequences of #2569 ... ;-) (with shoutouts to @alexarice for the #1156 redefinition of |
See also: my recent annotations to #2569 identifying possible inlining/deprecation opportunities. |
Now that #2568 / #2569 establish that (for
Setoid
s)Bijection
andInverse
are equivalent concepts, we should perhaps do a wholesale refactoring of theFunction
hierarchy to (choose and) deprecate whichever definitions (and theirPropositional
friends) we now deem to be redundant? See also #2565 .Revised: I had previously badged this as v2.3, but I suspect the effects will be so large-scale that it had probably better be tackled as a v3.0 series of tasks... not least the cleanup of the
*WithoutCongruent
definitions introduced by #2569 to restore the 'old' names, but now with stronger/sharper types than before.The text was updated successfully, but these errors were encountered: