<div dir="ltr"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><span style="font-size:12.8px">Yet I still see lots of code in that PR like:</span><br style="font-size:12.8px"><br style="font-size:12.8px"><span style="font-size:12.8px">if defined(GDAL_VERSION_NUM) && GDAL_VERSION_NUM >= GDAL_COMPUTE_VERSION(2,0,0)</span><br style="font-size:12.8px"><br style="font-size:12.8px"><span style="font-size:12.8px">That's what I'm referring to. All this conditional code needs removal</span><br style="font-size:12.8px"><span style="font-size:12.8px">(unless it's testing for something in GDAL > 2.0 ).</span></blockquote><div><br></div><div>Mosty in the static functions</div><div>- QgsOgrProviderUtils, mostly with a switch</div><div><br></div><div>and a few elsewhere.</div><div><br></div><div>There are also few for much older versions</div><div>#if defined(GDAL_VERSION_NUM) && GDAL_VERSION_NUM >= 1400</div><div><br></div><div>They are easy to remove.</div><div><br></div><div>There is one (at least) that must remain</div><div><br></div><div>#if defined(GDAL_COMPUTE_VERSION) && GDAL_VERSION_NUM >= GDAL_COMPUTE_VERSION(2,1,2)<br></div><div><br></div><div>I count 20, beforehand there were many more.</div><div><br></div><div>There also other classes sped around the project that uses OGR</div><div>- some of which also have these</div><div><br></div><div>Mark</div></div>