<div dir="ltr">Inlining for review.<div>Please note that both patches in this series are missing your "signed-off by" line.<br></div><div><br></div><div>Also, the issues addressed in this patch are unrelated and should be fixed separately.</div>
<div><br></div>
<div><div><div>> From c3474a73eb137b2bb7984d3195094a263c024e65 Mon Sep 17 00:00:00 2001</div><div>> From: Amit Margalit <<a href="mailto:amitm@il.ibm.com">amitm@il.ibm.com</a>></div><div>> Date: Mon, 19 Aug 2013 14:08:40 +0300</div>
<div>> Subject: [PATCH 1/2] Added missing call to _bt_ctf_get_decl_from_def in</div><div>>  get_int_len(), and defined ssize_t</div><div>> </div><div>> ---</div><div>>  bindings/python/<a href="http://babeltrace.i.in">babeltrace.i.in</a> | 3 ++-</div>
<div>>  1 file changed, 2 insertions(+), 1 deletion(-)</div><div>> </div><div>> diff --git a/bindings/python/<a href="http://babeltrace.i.in">babeltrace.i.in</a> b/bindings/python/<a href="http://babeltrace.i.in">babeltrace.i.in</a></div>
<div>> index dd2dacc..6c1f6fb 100644</div><div>> --- a/bindings/python/<a href="http://babeltrace.i.in">babeltrace.i.in</a></div><div>> +++ b/bindings/python/<a href="http://babeltrace.i.in">babeltrace.i.in</a></div>
<div>> @@ -53,6 +53,7 @@ trace to it."</div><div>>  typedef unsigned long long uint64_t;</div><div>>  typedef long long int64_t;</div><div>>  typedef int bt_intern_str;</div><div>> +typedef unsigned long ssize_t;</div>
<div><br></div><div>On which system is ssize_t undefined?</div><div>I'd personally use ptrdiff_t, but maybe it is undefined there too?</div><div><br></div><div>These typedefs should be handled at config time, is it something you would be interested in providing a patch for?</div>
<div><br></div><div>>  </div><div>>  /* =================================================================</div><div>>  <span class="" style="white-space:pre">               </span>CONTEXT.H, CONTEXT-INTERNAL.H</div><div>> @@ -866,7 +867,7 @@ class ctf:</div>
<div>>  <span class="" style="white-space:pre">                      </span>Return the size, in bits, of an int or a negative</div><div>>  <span class="" style="white-space:pre">                    </span>value on error.</div><div>>  <span class="" style="white-space:pre">                      </span>"""</div>
<div>> -<span class="" style="white-space:pre">                      </span>return _bt_ctf_get_int_len(self._d)</div><div>> +<span class="" style="white-space:pre">                  </span>return _bt_ctf_get_int_len(_bt_ctf_get_decl_from_def(self._d))</div>
<div><br></div><div>Good catch!</div><div><br></div><div>>  </div><div>>  <span class="" style="white-space:pre">             </span>def get_encoding(self):</div><div>>  <span class="" style="white-space:pre">                      </span>"""</div>
<div>> -- </div><div>> 1.7.11.7</div><div>> </div></div></div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Mon, Aug 19, 2013 at 7:15 AM, Amit Margalit <span dir="ltr"><<a href="mailto:AMITM@il.ibm.com" target="_blank">AMITM@il.ibm.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><font face="sans-serif">In some systems, the system headers use
ssize_t, which was not defined, causing get_int_len() to return a pointer
to a python object instead of returning the integer.</font>
<br><font face="sans-serif">Also, the python code that called _bt_ctf_get_int_len()
failed to wrap self._d with the call to _bt_ctf_get_decl_from_def.</font>
<br>
<br><font face="sans-serif">I'm attaching this patch to (hopefully)
avoid line formatting issues.</font>
<br><span class="HOEnZb"><font color="#888888">
<br>
<br>
<br>
<br><font color="#000080" face="sans-serif">Amit Margalit</font>
<br><font color="#808000" face="sans-serif">IBM XIV </font><font face="sans-serif">-
<i>Storage Reinvented</i></font>
<br><font face="sans-serif">XIV-NAS Development Team</font>
<br><font face="sans-serif">Tel. 03</font><font face="Arial">-689-7774</font>
<br><font face="Arial">Fax. 03-689-7230</font></font></span><br>_______________________________________________<br>
lttng-dev mailing list<br>
<a href="mailto:lttng-dev@lists.lttng.org">lttng-dev@lists.lttng.org</a><br>
<a href="http://lists.lttng.org/cgi-bin/mailman/listinfo/lttng-dev" target="_blank">http://lists.lttng.org/cgi-bin/mailman/listinfo/lttng-dev</a><br>
<br></blockquote></div><br><br clear="all"><div><br></div>-- <br>Jérémie Galarneau<br>EfficiOS Inc.<br><a href="http://www.efficios.com" target="_blank">http://www.efficios.com</a>
</div>