Re: Speedup hack enable/disable

Richard Low <[email protected]> Tue, 27 Sep 2005 11:34:25 +0100

Newsgroups	gmane.comp.multimedia.libnjb.user
Message-ID	<[email protected]>

On 26 Sep 2005, at 21:13, Linus Walleij wrote:

> OK the CVS turbo code is now controllable with an optional call.

Looks good (much better than my implementation), except I think you  
forgot to put the change in send_file() in procedure.c.  The attached  
patch (against current cvs) adds this and also can we replace the 0s  
and 1s with NJB_TURBO_OFF and NJB_TURBO_ON?

Thanks,

Richard.

procedure.c.diff (application/octet-stream, 388 B)

1148c1148
<       if (state->turbo_mode == NJB_TURBO_OFF) {
---
>       if (state->turbo_mode == 0) {
1356,1361c1356
< /*
<  * This speed-up hack works on most devices but is
<  * of course user-configurable.
< */
< njb3_state_t *state = (njb3_state_t *) njb->protocol_state;
< if (state->turbo_mode == NJB_TURBO_OFF)
---
> #ifdef NO_TURBO
1363c1358
< else
---
> #else
1369a1365
> #endif

protocol3.c.diff (application/octet-stream, 75 B)

75c75
<   state->turbo_mode = NJB_TURBO_ON;
---
>   state->turbo_mode = 1;