No, I had the same question too. This should be included to the body of the
function, no need to return something.


On 23 February 2014 08:41, David Ketcheson <notifications@github.com> wrote:

> In src/pyclaw/sharpclaw/solver.py:
>
> >          for i in xrange(nregisters-1):
> >              #Maybe should use State.copy() here?
> > -            self._rk_stages.append(State(state.patch,state.num_eqn,state.num_aux))
> > -            self._rk_stages[-1].problem_data       = state.problem_data
> > -            self._rk_stages[-1].set_num_ghost(self.num_ghost)
> > -            self._rk_stages[-1].t                = state.t
> > -            if state.num_aux > 0:
> > -                self._rk_stages[-1].aux              = state.aux
> > +            self._registers.append(State(state.patch,state.num_eqn,state.num_aux))
> > +            self._registers[-1].problem_data                = state.problem_data
> > +            self._registers[-1].set_num_ghost(self.num_ghost)
> > +            self._registers[-1].t                           = state.t
> > +            if state.num_aux > 0: self._registers[-1].aux   = state.aux
> > +
> > +        return self.sspcoeff[self.time_integrator]
>
> I don't think this function should return anything. Is the return value
> used somewhere?
>
> --
> Reply to this email directly or view it on GitHub<https://github.com/clawpack/pyclaw/pull/370/files#r9974460>
> .
>



-- 
Yiannis Hadjimichael