This is the mail archive of the gdb-patches@sourceware.org mailing list for the GDB project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [RFC/RFA] Add support for catch Ada exceptions (take 2)


Joel Brobecker <brobecker@adacore.com> writes:

>> > +  switch (b->type)
>> > +    {
>> > +      case bp_catch_exception:
>> > +        if (b->addr_string != NULL)
>> > +          {
>> > +            const char *template = _("`%s' Ada exception");
>> > +            char *msg = alloca (strlen (template) + strlen (b->addr_string));
>> > +            
>> > +            sprintf (msg, _("`%s' Ada exception"), b->addr_string);
>> 
>> Why don't you use `template' instead of having two identical strings?
>
> That's what I first did, but the compiler complains that he cannot check
> the string format.

If you use static const char[] instead then the compiler is able to do the
checks.

Andreas.

-- 
Andreas Schwab, SuSE Labs, schwab@suse.de
SuSE Linux Products GmbH, Maxfeldstraße 5, 90409 Nürnberg, Germany
PGP key fingerprint = 58CA 54C7 6D53 942B 1756  01D3 44D5 214B 8276 4ED5
"And now for something completely different."


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]