On Wed, 2014-02-05 at 12:02 -0800, Josh Stone wrote:
> On 02/05/2014 11:58 AM, Josh Stone wrote:
> > Also, an assertion on the range of line/column is now a handled error,
> > setting DWARF_E_INVALID_DWARF for values greater than INT_MAX.
> 
> Note, this is not so much INVALID as UNSUPPORTED, really, as the current
> API can only deal with int values.

2^31-1 should be enough for anybody :) There is probably really
something wrong when the DWARF indicates you have more than so many
source lines in a file. I think INVALID is correct in this case.

> I still think an assert was the wrong way to handle this

Agreed.

Cheers,

Mark

Reply via email to