<p>Good point with the ipxe.pxe sizes - however since the overall difference is just a few bytes we may observe unrelated compilation side effects. Also, the compiler version you are using appears to produce slightly larger binaries than mine - the "https master" version of ipxe.pxe is ~2kB larger than my "https this branch" one.</p>
<p>Another way of measuring the potential size increase would be to look at the ocsp.o binary and compare "https master" against "https this branch". ocsp.c is the only file touched by my PR. To prevent measuring unrelated compilation artifacts we would strip the binaries of debugging info.</p>
<pre><code>https master:
ls -la bin/ocsp.o
66100 bin/ocsp.o
strip bin/ocsp.o
ls -la bin/ocsp.o
 4696 bin/ocsp.o

https this branch:
ls -la bin/ocsp.o
73576 bin/ocsp.o
strip bin/ocsp.o
ls -la bin/ocsp.o
 5020 bin/ocsp.o
</code></pre>
<p>That's 324 more bytes.</p>

<p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">—<br />You are receiving this because you are subscribed to this thread.<br />Reply to this email directly, <a href="https://github.com/ipxe/ipxe/pull/90#issuecomment-464350482">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/AArTVL816BeaRwaJ-V-qjIQNBmBQmEumks5vOBKcgaJpZM4a-EdD">mute the thread</a>.<img src="https://github.com/notifications/beacon/AArTVCqeaulhMg9ekbVg8EA5OssCpZJfks5vOBKcgaJpZM4a-EdD.gif" height="1" width="1" alt="" /></p>
<script type="application/json" data-scope="inboxmarkup">{"api_version":"1.0","publisher":{"api_key":"05dde50f1d1a384dd78767c55493e4bb","name":"GitHub"},"entity":{"external_key":"github/ipxe/ipxe","title":"ipxe/ipxe","subtitle":"GitHub repository","main_image_url":"https://github.githubassets.com/images/email/message_cards/header.png","avatar_image_url":"https://github.githubassets.com/images/email/message_cards/avatar.png","action":{"name":"Open in GitHub","url":"https://github.com/ipxe/ipxe"}},"updates":{"snippets":[{"icon":"PERSON","message":"@t-lo in #90: Good point with the ipxe.pxe sizes - however since the overall difference is just a few bytes we may observe unrelated compilation side effects. Also, the compiler version you are using appears to produce slightly larger binaries than mine - the \"https master\" version of ipxe.pxe is ~2kB larger than my \"https this branch\" one.\r\n\r\nAnother way of measuring the potential size increase would be to look at the ocsp.o binary and compare \"https master\" against \"https this branch\". ocsp.c is the only file touched by my PR. To prevent measuring unrelated compilation artifacts we would strip the binaries of debugging info.\r\n```\r\nhttps master:\r\nls -la bin/ocsp.o\r\n66100 bin/ocsp.o\r\nstrip bin/ocsp.o\r\nls -la bin/ocsp.o\r\n 4696 bin/ocsp.o\r\n\r\nhttps this branch:\r\nls -la bin/ocsp.o\r\n73576 bin/ocsp.o\r\nstrip bin/ocsp.o\r\nls -la bin/ocsp.o\r\n 5020 bin/ocsp.o\r\n```\r\nThat's 324 more bytes."}],"action":{"name":"View Pull Request","url":"https://github.com/ipxe/ipxe/pull/90#issuecomment-464350482"}}}</script>
<script type="application/ld+json">[
{
"@context": "http://schema.org",
"@type": "EmailMessage",
"potentialAction": {
"@type": "ViewAction",
"target": "https://github.com/ipxe/ipxe/pull/90#issuecomment-464350482",
"url": "https://github.com/ipxe/ipxe/pull/90#issuecomment-464350482",
"name": "View Pull Request"
},
"description": "View this Pull Request on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
}
]</script>