Re: [svn:parrot] r17107 - trunk/src/stm

2007-02-23 Thread chromatic
On Thursday 22 February 2007 00:28, you wrote: > chromatic wrote: > >> +if (saved_log_data == NULL) > >> +return; > >> + > >> saved = saved_log_data; > >> mem_sys_free(saved->reads); > >> mem_sys_free(saved->writes); > > I'm not sure this is the right fix; there's litt

Re: [svn:parrot] r17107 - trunk/src/stm

2007-02-23 Thread jerry gay
On 2/22/07, Allison Randal <[EMAIL PROTECTED]> wrote: 'mark_extracted' should have the guard added. The same one should work. As intended, it is marking all the read_records and write_records. There just happen to be none to mark. added in r17159, thanks. ~jerry

Re: [svn:parrot] r17107 - trunk/src/stm

2007-02-22 Thread Allison Randal
chromatic wrote: +if (saved_log_data == NULL) +return; + saved = saved_log_data; mem_sys_free(saved->reads); mem_sys_free(saved->writes); I'm not sure this is the right fix; there's little reason to use an STMLog PMC if it doesn't get a valid transaction log. My vo

Re: [svn:parrot] r17107 - trunk/src/stm

2007-02-20 Thread chromatic
On Tuesday 20 February 2007 19:01, [EMAIL PROTECTED] wrote: > Modified: >trunk/src/stm/backend.c > > Log: > [stm]: don't access a null pointer > > Modified: trunk/src/stm/backend.c > === >=== --- trunk/src/stm/backend.c