[PATCH 1/3 v4] Introduce common/errors.h

Doug Evans dje@google.com
Thu Jul 24 18:03:00 GMT 2014


Pedro Alves writes:
 > > +   You should have received a copy of the GNU General Public License
 > > +   along with this program.  If not, see <http://www.gnu.org/licenses/>.  */
 > > +
 > > +#include "config.h"
 > 
 > Sorry for the push back, but it's wrong to only include "config.h".
 > We need to include gnulib's config.h everywhere too.  I think
 > the best is to add a central common header that handles that
 > detail first.

OTOH,
just including "config.h" is a real common thing to do across all packages.
I can imagine this coming up again and again.

IWBN if hacking on gdb didn't require special cases to standard
programming paradigms wherever possible.
Can we arrange for config.h to include gnulib's config.h?
Or, another thought would be to have configure generate gdb-config.h
(or some such) and then have src/gdb/config.h include both (and similarly
for gdbserver - haven't looked at the details though).



More information about the Gdb-patches mailing list