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 > > > > > -- Mailing list: https://launchpad.net/~oqgraph-dev Post to : oqgraph-dev@lists.launchpad.net Unsubscribe : https://launchpad.net/~oqgraph-dev More help : https://help.launchpad.net/ListHelp
