Rockbox

This is a good patch.
Just one question.
When .talk clips are enabled can it speak the file exstention?
So for example if I scrol over a text file called file.txt can it say file from the .talk clip and then say .txt from the voice file.

But I think this is a really useful patch. The only thing that might stop it been committed is the extra setting. I don't think it is too much so I can't see why it can't be committed.
Thanks.

This is a good patch.
>Just one question.
>When .talk clips are enabled can it speak the file exstention?
>So for example if I scrol over a text file called file.txt can it say file
>from the .talk clip and then say .txt from the voice file.

What would be the benefit?

My thinking was that the .talk clip already contains the voicing of
the extension. We could strip the extension when generating the .talk clips,
and speak it from the voice file, but there are disadvantages:
-There would probably be a slight delay in switching from speaking the .talk
clip to the vice file entry,
-Not all extensions are defined as file types, for example it seems .txt
isn't. We have to fallback to spelling which is slower.

But I think this is a really useful patch. The only thing that might stop it
>been committed is the extra setting. I don't think it is too much so I can't
>see why it can't be committed.

Well I like it enabled myself and wouldn't mind not having a setting at all,
but I added the option because there seemed to be some concerns that it could
be confusing in the previous incarnation of the patch.

Well here on voice box if I have a file called file.txt the voice clip says "File" then I was thinking we could have it say ".txt" so it would say "file.txt". But I can see what your saying about the delay. And I don't think one setting should be to much. Maybe we just need to bring it up in irc and hopefully get it committed.

O yeah the .talk file would be called file.txt.talk.

Oh I see… I had been using a script of my own to generate .talk
clips, and I hadn't realized that the "official" script strips file
extensions by default.

tools/gentalkclips.sh has this line:

  STRIP_EXTENSIONS=Y

I suppose some people might find it mor aesthetically pleasing not to
hear the extension… myself I'd rather just hear what's there. There are
settings to configure what gets displayed: for example, sighted users
can have the extensions stripped (possibly with the exception of
unrecognized ones) and have an icon shown instead.

Note that the spoken filetypes, like the icons, are more generic: it
says "audio" rather than .mp3 or .ogg or whatever. I don't think we'd
want to add more voice entries for each possible extension.

Would you want to hear "title of this song, audio",
or "title of this song dot mp3"? The second is more informative and less
confusing. Or put the type first, but that slows down browsing in a
directory full of audio files… It's good in the spelling case, not in
the .talk clips case.

I would recommend to just keep this simple: just don't strip the
extensions from the spoken name in .talk clips. Thoughts?

This version applies on top of P#7775: Use list voice callback in file browser.

   say-filetype.diff (3.8 KiB)

Ok I modified my copy of the voicebox script to not strip the file extensions. So don't worry about my other comments.

Updated version. I intend to commit this soon if there are no further
comments.

I have had at least one additional positive comment by private mail. It
allows to quickly skip items of the wrong type and to hear less
spelling.

Changes:

* Do not say the file type when browsing files of a specific type, as

it's redundant.

* Get rid of the setting. Let's try not to add too many new settings,

unless someone really dislikes this feature. I know two users have said
on the list that they see no reason to remove the setting, but no
argument has been made that disabling this is particularly useful.

   filetype.diff (1.1 KiB)