| … | … | @@ -178,8 +178,13 @@ static ValuePtr exec_or_eval(AtomSpace* as, |
| 178 | 178 | vp = hs[0]; |
| 179 | 179 | } |
| 180 | 180 | |
| 181 | | // XXX FIXME? should we be doing the add here? Isn't this the |
| 182 | | // callers reponsibility? |
| 181 | // XXX FIXME? Should we be doing the add here? Isn't this the |
| 182 | // callers reponsibility? Three unit tests fail, if we don't |
| 183 | // do the add: RandomUTest FormulaUTest SequenceUTest and one |
| 184 | // fails: SequenceUTest if we add to scratch instead of main. |
| 185 | // I'm not sure if something is buggy or not, or if this is an |
| 186 | // intentional design choice. FWIW there are many places in the |
| 187 | // code base that do something like this. |
| 183 | 188 | // if (vp->is_atom()) scratch->add_atom(HandleCast(vp)); |
| 184 | 189 | if (vp->is_atom()) as->add_atom(HandleCast(vp)); |
| 185 | 190 | return vp; |