<div><span style="line-height: 1.5;">Hi,</span></div><div><br></div><div>Today I build latest xapian from git, and find some errors.</div><div><span style="line-height: 1.5;">For example, in xapian-core/api/valueiterator.cc, Line 141,</span></div><div><span style="line-height: 1.5;"><br></span></div><div><ol class="linenums gts-lines" style="padding: 0px; margin: 0px 0px 0px 40px; list-style-position: initial; list-style-image: initial; color: rgb(51, 51, 51); font-family: Menlo, Monaco, Consolas, 'Courier New', monospace; font-size: 12px; line-height: 18px; white-space: pre-wrap;"><li class="L130" style="line-height: 18px; color: rgb(190, 190, 197); padding-left: 12px; text-shadow: rgb(255, 255, 255) 0px 1px 0px;"><span class="line" style="color: rgb(72, 72, 76); display: block; min-height: 18px;"><span class="kt" style="color: rgb(176, 0, 64);">bool</span></span></li><li class="L131" style="line-height: 18px; color: rgb(190, 190, 197); padding-left: 12px; text-shadow: rgb(255, 255, 255) 0px 1px 0px;"><span class="line" style="color: rgb(72, 72, 76); display: block; min-height: 18px;"><span class="n">ValueIterator</span><span class="o" style="color: rgb(102, 102, 102);">::</span><span class="n">check</span><span class="p">(</span><span class="n">Xapian</span><span class="o" style="color: rgb(102, 102, 102);">::</span><span class="n">docid</span> <span class="n">did</span><span class="p">)</span></span></li><li class="L132" style="line-height: 18px; color: rgb(190, 190, 197); padding-left: 12px; text-shadow: rgb(255, 255, 255) 0px 1px 0px;"><span class="line" style="color: rgb(72, 72, 76); display: block; min-height: 18px;"><span class="p">{</span></span></li><li class="L133 focus" style="line-height: 18px; color: rgb(72, 72, 78); padding-left: 12px; text-shadow: rgb(255, 255, 255) 0px 1px 0px; border-right-width: 1px; border-right-style: solid; border-right-color: rgb(225, 225, 232);"><span class="line" style="color: rgb(72, 72, 76); display: block; min-height: 18px; background-color: rgb(255, 255, 204); margin-left: -60px; padding-left: 60px; margin-right: -8px; padding-right: 8px; background-position: initial initial; background-repeat: initial initial;">    <span class="n">LOGCALL</span><span class="p">(</span><span class="n">API</span><span class="p">,</span> <span class="kt" style="color: rgb(176, 0, 64);">bool</span><span class="p">,</span> <span class="s" style="color: rgb(221, 17, 68);">"ValueIterator::check"</span><span class="p">,</span> <span class="n">did</span><span class="p">);</span></span></li><li class="L134" style="line-height: 18px; color: rgb(190, 190, 197); padding-left: 12px; text-shadow: rgb(255, 255, 255) 0px 1px 0px;"><span class="line" style="color: rgb(72, 72, 76); display: block; min-height: 18px;">    <span class="k" style="color: rgb(30, 52, 123); font-weight: bold;">if</span> <span class="p">(</span><span class="n">internal</span><span class="p">)</span> <span class="p">{</span></span></li><li class="L135" style="line-height: 18px; color: rgb(190, 190, 197); padding-left: 12px; text-shadow: rgb(255, 255, 255) 0px 1px 0px;"><span class="line" style="color: rgb(72, 72, 76); display: block; min-height: 18px;">    <span class="k" style="color: rgb(30, 52, 123); font-weight: bold;">if</span> <span class="p">(</span><span class="o" style="color: rgb(102, 102, 102);">!</span><span class="n">internal</span><span class="o" style="color: rgb(102, 102, 102);">-></span><span class="n">check</span><span class="p">(</span><span class="n">did</span><span class="p">))</span> <span class="k" style="color: rgb(30, 52, 123); font-weight: bold;">return</span> <span class="nb" style="color: rgb(30, 52, 123); font-weight: bold;">false</span><span class="p">;</span></span></li><li class="L136" style="line-height: 18px; color: rgb(190, 190, 197); padding-left: 12px; text-shadow: rgb(255, 255, 255) 0px 1px 0px;"><span class="line" style="color: rgb(72, 72, 76); display: block; min-height: 18px;">    <span class="k" style="color: rgb(30, 52, 123); font-weight: bold;">if</span> <span class="p">(</span><span class="n">internal</span><span class="o" style="color: rgb(102, 102, 102);">-></span><span class="n">at_end</span><span class="p">())</span> <span class="p">{</span></span></li><li class="L137" style="line-height: 18px; color: rgb(190, 190, 197); padding-left: 12px; text-shadow: rgb(255, 255, 255) 0px 1px 0px;"><span class="line" style="color: rgb(72, 72, 76); display: block; min-height: 18px;">        <span class="n">decref</span><span class="p">();</span></span></li><li class="L138" style="line-height: 18px; color: rgb(190, 190, 197); padding-left: 12px; text-shadow: rgb(255, 255, 255) 0px 1px 0px;"><span class="line" style="color: rgb(72, 72, 76); display: block; min-height: 18px;">        <span class="n">internal</span> <span class="o" style="color: rgb(102, 102, 102);">=</span> <span class="nb" style="color: rgb(30, 52, 123); font-weight: bold;">NULL</span><span class="p">;</span></span></li><li class="L139" style="line-height: 18px; color: rgb(190, 190, 197); padding-left: 12px; text-shadow: rgb(255, 255, 255) 0px 1px 0px;"><span class="line" style="color: rgb(72, 72, 76); display: block; min-height: 18px;">    <span class="p">}</span></span></li><li class="L140" style="line-height: 18px; color: rgb(190, 190, 197); padding-left: 12px; text-shadow: rgb(255, 255, 255) 0px 1px 0px;"><span class="line" style="color: rgb(72, 72, 76); display: block; min-height: 18px;">    <span class="p">}</span></span></li><li class="L141" style="line-height: 18px; color: rgb(190, 190, 197); padding-left: 12px; text-shadow: rgb(255, 255, 255) 0px 1px 0px;"><span class="line" style="color: rgb(72, 72, 76); display: block; min-height: 18px;">    <span class="k" style="color: rgb(30, 52, 123); font-weight: bold;">return</span> <span class="nb" style="color: rgb(30, 52, 123); font-weight: bold;">true</span><span class="p">;</span></span></li><li class="L142" style="line-height: 18px; color: rgb(190, 190, 197); padding-left: 12px; text-shadow: rgb(255, 255, 255) 0px 1px 0px;"><span class="line" style="color: rgb(72, 72, 76); display: block; min-height: 18px;"><span class="p">}</span></span></li></ol><div><font color="#48484c" face="Menlo, Monaco, Consolas, Courier New, monospace"><span style="font-size: 12px; line-height: 18px; white-space: pre-wrap;"><br></span></font></div></div><div><font color="#48484c" face="Menlo, Monaco, Consolas, Courier New, monospace"><span style="font-size: 12px; line-height: 18px; white-space: pre-wrap;"><br></span></font></div><div><font color="#48484c" face="Menlo, Monaco, Consolas, Courier New, monospace"><span style="font-size: 12px; line-height: 18px; white-space: pre-wrap;">LOGCALL is used, so we should use RETURN rather than return.</span></font></div><div><font color="#48484c" face="Menlo, Monaco, Consolas, Courier New, monospace"><span style="font-size: 12px; line-height: 18px; white-space: pre-wrap;">Because of the use of "return", my complier give a warning </span></font></div><div><font color="#48484c" face="Menlo, Monaco, Consolas, Courier New, monospace"><span style="font-size: 12px; line-height: 18px; white-space: pre-wrap;"><br></span></font></div><div><font color="#48484c" face="Menlo, Monaco, Consolas, Courier New, monospace"><span style="font-size: 12px; line-height: 18px; white-space: pre-wrap;">typedef 'xapian_logcall_return_type_' locally defined but not used [-Werror=unused-local-typedefs]
     typedef TYPE xapian_logcall_return_type_; \</span></font></div><div><font color="#48484c" face="Menlo, Monaco, Consolas, Courier New, monospace"><span style="font-size: 12px; line-height: 18px; white-space: pre-wrap;"><br></span></font></div><div><font color="#48484c" face="Menlo, Monaco, Consolas, Courier New, monospace"><span style="font-size: 12px; line-height: 18px; white-space: pre-wrap;">which is regarded as an error, so I can get it built.</span></font></div><div><font color="#48484c" face="Menlo, Monaco, Consolas, Courier New, monospace"><span style="font-size: 12px; line-height: 18px; white-space: pre-wrap;"><br></span></font></div><div><font color="#48484c" face="Menlo, Monaco, Consolas, Courier New, monospace"><span style="font-size: 12px; line-height: 18px; white-space: pre-wrap;">And it seems there are lots of same errors, e.g. in omdatabase.cc, Line 126</span></font></div><div><font color="#48484c" face="Menlo, Monaco, Consolas, Courier New, monospace"><span style="font-size: 12px; line-height: 18px; white-space: pre-wrap;">omdocument.cc, omenquire.cc, queryinternal.cc ....and so on...</span></font></div><font color="#48484c" face="Menlo, Monaco, Consolas, Courier New, monospace"><span style="font-size: 12px; line-height: 18px; white-space: pre-wrap;"> </span></font><div><font color="#48484c" face="Menlo, Monaco, Consolas, Courier New, monospace"><span style="font-size: 12px; line-height: 18px; white-space: pre-wrap;"><br></span></font><div><br></div><div><br></div><div><div style="color:#909090;font-family:Arial Narrow;font-size:12px">------------------</div><div style="font-size:14px;font-family:Verdana;color:#000;">Shangtong Zhang,<div>Second Year Undergraduate,</div><div>School of Computer Science,</div><div>Fudan University, China.</div></div></div><div> </div></div>