oqgraph-dev team mailing list archive

[Andrew McDonnell <bugs@xxxxxxxxxxxxxxxxxxx>]

Having said that, I still found more crashes further down if you do something
silly like creating the table with   origid=''


So there are quite a few edge test cases, I will need to make a table of them all


One thing I have realised, is that these conditions are only detected when
attempting to insert, it would be useful to validate the attributes when the
table is created, except you probably need to be able to support having the
source data table be dropped and recreated so they probably need to be
warnings rather than fails


--Andrew


On 26/02/13 21:07, Andrew McDonnell wrote:
> Hiya
> 
> the first patch checks the attributes are present as expected without crashing
> the daemon. I have tried to be consistent with the error messages.
> 
> the second patch fixes an assertion that occurred with DEBUG enabled where the
> length of the table was off by one for some reason
> 
> I probably need to work out how to push this back to launchpad, I think you
> mentioned it in an earlier email
> 
> --Andrew
> 
> 
> 
> 
>