speakup-r functionality

[Okash Khawaja]

Okay I tested with the file you sent and as you described, but
couldn't replicate. If it does behave differently with serial synth
then it _could_ be that another issue has been exposed by this change.
I don't have a functional serial synth which supports indexing. I will
look into fixing my doubletalk lt.

Just to be sure, following is the patch which breaks speakup-r?

---
 drivers/staging/speakup/main.c |    3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

--- a/drivers/staging/speakup/main.c
+++ b/drivers/staging/speakup/main.c
@@ -1408,7 +1408,8 @@ static void read_all_doc(struct vc_data
  cursor_track = read_all_mode;
  spk_reset_index_count(0);
  if (get_sentence_buf(vc, 0) == -1) {
- kbd_fakekey2(vc, RA_DOWN_ARROW);
+ del_timer(&cursor_timer);
+ start_read_all_timer(vc, RA_DOWN_ARROW);
  } else {
  say_sentence_num(0, 0);
  synth_insert_next_index(0);

Thanks,
Okash

On Sun, Aug 20, 2017 at 5:12 PM, John Covici <covici at ccs.covici.com> wrote:
> In my latest test, it still did not work.  I will send you a file to
> test with.  I started  at the beginning of the file, went to line 25
> and hit control and the cursor was at line 198.  Now it might work
> differently with speech dispatcher, you probably should use a serial
> synth to test.  Without the speakup-r empty line fix, it does work.
>
> On Mon, 29 May 2017 05:50:33 -0400,
> Okash Khawaja wrote:
>>
>> On Sun, May 28, 2017 at 06:38:39PM -0400, John Covici wrote:
>> > I just have one question -- how did it work with the old serial i/o
>> > system?
>> In main.c, handle_cursor_read_all calls spk_get_index_count which calls
>> spk_ttyio_in_nowait or spk_serial_in_nowait depending on whether TTY or
>> old serial i/o is in use. So the difference is in spk_serial_in_nowait
>> as the layers above that are identical. spk_serial_in_nowait returns 0
>> when there is no data on rx. spk_ttyio_in_nowait immitates that but due
>> to the signedness bug, it was returning 0xff instead of 0 when there
>> is no data.
>>
>> With the fix, now both serial i/o and TTY stacks  should have same
>> behaviour.
>
> --
> Your life is like a penny.  You're going to lose it.  The question is:
> How do
> you spend it?
>
>          John Covici
>          covici at ccs.covici.com