Skip to content
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

Code Constification #1300

Closed
wants to merge 14 commits into from
Closed

Conversation

FdaSilvaYY
Copy link
Contributor

I try to be conservative ;)
Feel free to comment

typedef ASN1_VALUE *ASN1_new_func(void);
typedef void ASN1_free_func(ASN1_VALUE *a);
typedef ASN1_VALUE *ASN1_d2i_func(ASN1_VALUE **a, const unsigned char **in,
long length);
typedef int ASN1_i2d_func(ASN1_VALUE *a, unsigned char **in);

*/
Copy link
Member

Choose a reason for hiding this comment

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

Why do you comment them out?

Copy link
Contributor

Choose a reason for hiding this comment

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

yes, please back this change out from this MR.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes, it is out-of-topic., my mistake.

@FdaSilvaYY FdaSilvaYY force-pushed the ossl_constification branch 7 times, most recently from 022f9c5 to 96dff3e Compare July 17, 2016 23:01
@mattcaswell mattcaswell added this to the 1.1.0 milestone Jul 18, 2016
@FdaSilvaYY FdaSilvaYY force-pushed the ossl_constification branch 5 times, most recently from 0bc16c3 to 8f572ba Compare July 21, 2016 21:07
@richsalz richsalz self-assigned this Jul 22, 2016
@richsalz
Copy link
Contributor

+1; needs a second review.

levitte pushed a commit that referenced this pull request Jul 25, 2016
simplify and reindent some related code.

Reviewed-by: Matt Caswell <matt@openssl.org>
Reviewed-by: Rich Salz <rsalz@openssl.org>
(Merged from #1300)
levitte pushed a commit that referenced this pull request Jul 25, 2016
Reviewed-by: Matt Caswell <matt@openssl.org>
Reviewed-by: Rich Salz <rsalz@openssl.org>
(Merged from #1300)
levitte pushed a commit that referenced this pull request Jul 25, 2016
Reviewed-by: Matt Caswell <matt@openssl.org>
Reviewed-by: Rich Salz <rsalz@openssl.org>
(Merged from #1300)
levitte pushed a commit that referenced this pull request Jul 25, 2016
Reviewed-by: Matt Caswell <matt@openssl.org>
Reviewed-by: Rich Salz <rsalz@openssl.org>
(Merged from #1300)
levitte pushed a commit that referenced this pull request Jul 25, 2016
…YPE_get_octetstring & co...

Reviewed-by: Matt Caswell <matt@openssl.org>
Reviewed-by: Rich Salz <rsalz@openssl.org>
(Merged from #1300)
levitte pushed a commit that referenced this pull request Jul 25, 2016
Reviewed-by: Matt Caswell <matt@openssl.org>
Reviewed-by: Rich Salz <rsalz@openssl.org>
(Merged from #1300)
levitte pushed a commit that referenced this pull request Jul 25, 2016
Reviewed-by: Matt Caswell <matt@openssl.org>
Reviewed-by: Rich Salz <rsalz@openssl.org>
(Merged from #1300)
levitte pushed a commit that referenced this pull request Jul 25, 2016
Reviewed-by: Matt Caswell <matt@openssl.org>
Reviewed-by: Rich Salz <rsalz@openssl.org>
(Merged from #1300)
levitte pushed a commit that referenced this pull request Jul 25, 2016
 - X509_NAME_entry_count, X509_ATTRIBUTE_count
 - X509_NAME_add_entry_by_OBJ, X509_NAME_ENTRY_create_by_OBJ, X509_NAME_ENTRY_set_object

Reviewed-by: Matt Caswell <matt@openssl.org>
Reviewed-by: Rich Salz <rsalz@openssl.org>
(Merged from #1300)
levitte pushed a commit that referenced this pull request Jul 25, 2016
Reviewed-by: Matt Caswell <matt@openssl.org>
Reviewed-by: Rich Salz <rsalz@openssl.org>
(Merged from #1300)
levitte pushed a commit that referenced this pull request Jul 25, 2016
Reviewed-by: Matt Caswell <matt@openssl.org>
Reviewed-by: Rich Salz <rsalz@openssl.org>
(Merged from #1300)
levitte pushed a commit that referenced this pull request Jul 25, 2016
X509_REVOKED_get0_extensions
X509_check_private_key

Reviewed-by: Matt Caswell <matt@openssl.org>
Reviewed-by: Rich Salz <rsalz@openssl.org>
(Merged from #1300)
levitte pushed a commit that referenced this pull request Jul 25, 2016
Reviewed-by: Matt Caswell <matt@openssl.org>
Reviewed-by: Rich Salz <rsalz@openssl.org>
(Merged from #1300)
levitte pushed a commit that referenced this pull request Jul 25, 2016
Reviewed-by: Matt Caswell <matt@openssl.org>
Reviewed-by: Rich Salz <rsalz@openssl.org>
(Merged from #1300)
@kaduk
Copy link
Contributor

kaduk commented Jul 25, 2016

(looks like some trailing whitespace snuck in in a couple of these commits, but "meh")

@richsalz
Copy link
Contributor

Yup. Meh.

Senior Architect, Akamai Technologies
IM: richsalz@jabber.at Twitter: RichSalz

From: kaduk [mailto:notifications@github.com]
Sent: Monday, July 25, 2016 11:42 AM
To: openssl/openssl
Cc: Salz, Rich; State change
Subject: Re: [openssl/openssl] Code Constification (#1300)

(looks like some trailing whitespace snuck in in a couple of these commits, but "meh")


You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHubhttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_openssl_openssl_pull_1300-23issuecomment-2D234992068&d=DQMCaQ&c=96ZbZZcaMF4w0F4jpN6LZg&r=4LM0GbR0h9Fvx86FtsKI-w&m=mJ72Hd5aSd3oI3i97EpcIiShC4g1E-_ZM2gsP0GDPIU&s=NagDID0fXWrsYXcd_KTzOkpw03hOQItC64w-wPq-wK4&e=, or mute the threadhttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_notifications_unsubscribe-2Dauth_AG-5FilHdcTJ7rKvVm8N6z5MPNOyUtGzBEks5qZNlKgaJpZM4JJE-2DW&d=DQMCaQ&c=96ZbZZcaMF4w0F4jpN6LZg&r=4LM0GbR0h9Fvx86FtsKI-w&m=mJ72Hd5aSd3oI3i97EpcIiShC4g1E-_ZM2gsP0GDPIU&s=WmxSjlVO2Dz21tSiCimCmKQI8s0au8I-SlMlq_f4nuk&e=.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approval: done This pull request has the required number of approvals
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants