-
Notifications
You must be signed in to change notification settings - Fork 15
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
779 hash function #937
779 hash function #937
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks, looks good otherwise.
hash functions with implementation-defined semantics.</p> | ||
</fos:rules> | ||
<fos:errors> | ||
<p>A dynamic error is raised ([TODO: error code]) if the effective value of |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
FOCH0005
might be an appropriate error code (unfortunately, the XQFO error codes are quite cryptic).
Thanks, Christian, for the excellent suggestions and catches. Latest commit addresses them all, + a few more. To-do / desiderata would be examples using binary input. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
…some more observations, all minor.
I still do not understand why this would be put into fn and not into a separate hash module. Or it could fit into Expath's binary module md5 and sha-1 are already deprecated anyways |
The CG agreed to merge this PR at meeting 062 |
First draft of hash function, proposed in #779.
Error message left as to-do item; guidance from editors appreciated.
I opted to leave out wrapper/cryptographic functionality, such as salting, and to demonstrate via example how it could be done by a developer on their own. In my opinion what we need here is a simple atomic function that can be incorporated into other molecular functions.
I may tinker with the prose description up to CG discussion, so comments are welcome.