<!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;}</style>
</head>
<body>
<p>Hi Steve,</p>
<p>On 2017-03-03 13:39, Steve Lhomme wrote:</p>
<blockquote style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;color:#500050">
<pre><code> On Fri, Mar 3, 2017 at 1:35 PM, Filip Roséen <filip@atch.se> wrote:</code></pre>
<blockquote style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;color:#500050">
<pre><code> Looks alright to be, but there is a separate define related to la_ssize_t;
 __LA_SSIZE_T_DEFINED. I think the end-result will be the same since I think
 that if one of the defines is missing, then both are.

 Have you checked if there’s any way of having either of __LA_INT64_T_DEFINED
 and __LA_SSIZE_T_DEFINED, but not both?</code></pre>
</blockquote>
<pre><code> No, I only checked a way to differentiate between old and new headers.
 They don't even have a version for the API/name changes...</code></pre>
</blockquote>
<p>There’s <code>ARCHIVE_VERSION_NUMBER</code>, which should be stable enough to make decision based on the <em>libarchive</em> version.</p>
<p>In either case, I think the safest bet is to use both of <code>__LA_SSIZE_T_DEFINED</code> and <code>__LA_INT64_T_DEFINED</code>, even though I guess using one (as you did) is probably good enough.</p>
</body>
</html>