<div dir="ltr">In Even's <a href="https://trac.osgeo.org/gdal/changeset/41094">https://trac.osgeo.org/gdal/changeset/41094</a>:<div><br></div><div>Even handles the multiple definitions of OGRRegisterAll().  I propose that we remove the one from ogrsf_frmts.h ( which was not my first thought ).  I think the common cases are:<div><br></div><div>1) you just want all the drivers registered and don't care about individual registers, so just use ogr_api.h.  You will likely need it anyway.</div><div><br></div><div>2) you are definiting a register function and don't care about OGRRegisterAll so include ogrsf_frmts.h</div><div><br></div><div>3) you want a specific driver registered and don't care about OGRRegisterAll so include ogrsf_frmts.h</div><div><br></div><div>Cases 2 and 3 likely will have to include ogr_api.h to do their work, but maybe not in a rare case.</div><div><br></div><div>Why add extra in case 1?</div></div><div><br></div><div>Thoughts?</div></div>