Skip to content

Commit

Permalink
checksums.openssl: make evp-md-context private.
Browse files Browse the repository at this point in the history
  • Loading branch information
@mrjbq7
mrjbq7 committed Jun 29, 2016
1 parent 0feed3c commit d9f1731
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions basis/checksums/openssl/openssl.factor
View file
Expand Up @@ -17,6 +17,8 @@ INSTANCE: openssl-checksum stream-checksum

C: <openssl-checksum> openssl-checksum

<PRIVATE

TUPLE: evp-md-context < disposable handle ;

: <evp-md-context> ( -- ctx )
Expand All @@ -26,8 +28,6 @@ TUPLE: evp-md-context < disposable handle ;
M: evp-md-context dispose*
handle>> EVP_MD_CTX_destroy ;

<PRIVATE

: digest-named ( name -- md )
dup EVP_get_digestbyname
[ ] [ unknown-digest ] ?if ;
Expand Down

4 comments on commit d9f1731

@bjourne
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not sure yet if this commit is the problem, but when I run load-all:

46: M: evp-md-context add-checksum-bytes ( ctx bytes -- ctx' )
                                        ^
Invalid parameters for create-method
class   evp-md-context
generic add-checksum-bytes

The following restarts are available:

@mrjbq7
Copy link
Member Author

@mrjbq7 mrjbq7 commented on d9f1731 Jun 30, 2016 •
edited

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hmm, I don't see that and the builders seem okay...

"checksums" load looks clean.

@mrjbq7
Copy link
Member Author

@mrjbq7 mrjbq7 commented on d9f1731 Jun 30, 2016

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oh, you need to refresh checksums vocabulary because the signature of add-checksum-bytes changed from a normal word to a generic one.

@bjourne
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ah you're right, sorry bout the noise!

Please sign in to comment.