<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">

<html xmlns="http://www.w3.org/1999/xhtml">

<head>

  <meta http-equiv="Content-Type" content="text/html; charset=utf-8" />

  <meta http-equiv="Content-Style-Type" content="text/css" />

  <meta name="generator" content="pandoc" />

  <title></title>

  <style type="text/css">

      code{white-space: pre-wrap;}

      span.smallcaps{font-variant: small-caps;}

      span.underline{text-decoration: underline;}

      div.column{display: inline-block; vertical-align: top; width: 50%;}

  </style>

</head>

<body>

<p>Hi Steve,</p>

<p>On 2018-07-11 11:11, Steve Lhomme wrote:</p>

<blockquote style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;color:#500050">

<pre><code> On 2018-07-11 11:03, Filip Roséen wrote:</code></pre>

<blockquote style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;color:#500050">

<pre><code> A cast to |uintptr_t| is necessary, and the format-specifier

 should be using |PRIuPTR|. Though, it is certainly debatable

 whether |uintptr_t| should be used in the code at all.</code></pre>

</blockquote>

<pre><code> I think intptr_t would be more logical to respect the original %zi.

 But on the other hand the p_sys value is always used as a uintptr_t

 in the rest of the code so I guess it would be better.</code></pre>

</blockquote>

<p>My vote is fixing the relevant path instead of putting fresh paint on a dirty road; the change in question did nothing to increase code-quality given the rest of the file.</p>

<p>I do however like that someone takes the time to fix the broken format-specifiers in the codebase, so you get a star for taking on such mind-numbing task.</p>

</body>

</html>