1
0
Fork 0
forked from len0rd/rockbox
Commit graph

17367 commits

Author SHA1 Message Date
William Wilgus
afba86f0f7 [coverity] playlist.c pl_save_update_control() close fd on error
Change-Id: Ib44e86a36540e65a15eed3e9b99106734379250d
2024-06-02 22:14:27 -04:00
William Wilgus
283f31dd39 [coverity] language.c lang_load() check bytes read for headers
Change-Id: I546f316692c6d0c201d36d3346aab7aab6cc4919
2024-06-02 22:07:02 -04:00
William Wilgus
ce6fbe4382 [coverity] cuesheet.c search_for_cursheet() if no dot strcpy to NULL
the logic here is a bit fuzzy to me maybe one of the copies to the cuepath
buffer adds a dot but I don't see anything explicit

Change-Id: Idcc469b5d04e4d9125b36df53d6aaa32a10f3bfd
2024-06-02 21:48:00 -04:00
William Wilgus
f4a044e561 [coverity] playlist.c get_track_filename() uninitialized buffer
format_track_path() uses path_append_ex() which might use strlen on the
existing buffer likely a false positive but cheap enough to guard against

Change-Id: I7061a434460f235fc58016f9dcf2dfe4633b9f59
2024-06-02 21:40:20 -04:00
William Wilgus
31ae252dcc [BugFix] Resume on headphone plug causes screen glitches
Change-Id: I396a662124a20420f6099522570c14cef4993205
2024-06-02 17:09:24 -04:00
William Wilgus
bbdba53413 [coverity] bookmark.c fix add_bookmark & false positive in remove_bookmark
Change-Id: Ifad76ac8cb297a8008127d4cd73056c5309167c1
2024-06-02 12:42:45 -04:00
William Wilgus
2e60316500 [coverity] skin_parser.c load_skin_bitmaps check buffer length for bm_data
Change-Id: Ia4496916bf32de332e054b62f1708b3e2ec8e7d8
2024-06-02 11:13:03 -04:00
William Wilgus
5c7fbe8f1b [coverity] tree.c set_current_file_ex explicitly check strrchr output
the math
        size_t endpos = filename - path;
        if (filename && endpos < MAX_PATH - 1)
should handle it just fine coverity doesn't think so

Change-Id: Ifa0494e56b40c8ab0081ebc771024a45cb24eb76
2024-06-02 10:41:50 -04:00
Solomon Peachy
5d34887d4a voice: regression: Don't mangle the filename when spelling it out.
26c612f breaks file selections if you have filename extensions disabled
and filenames spelled out, because it modified the filename in place
breaking the caller.

Instead of using a lot of space to make a private copy, the simplest
approach is, if we strip the extension off, simply restore it
afterwards.

Change-Id: Iaf560e813053b8030c620bbed6d061a2423338d5
2024-06-01 18:27:09 -04:00
Solomon Peachy
e16230de8c FS#13433: Updated German Translation (Wilfried Winkler)
Change-Id: I87bc658186d2b65caded0e7cde7b9c8110da000c
2024-06-01 15:07:03 -04:00
Christian Soffke
bcd34cfaa7 playlist catalogue (etc.): prune context menu
"Create Directory" option not applicable
in browse filter modes, since directories
do not get displayed

Change-Id: I68944264b03115f7f94254d9df766e5584d30e5a
2024-06-01 12:40:45 -04:00
Christian Soffke
507fa214a8 Fix red in 3d7d1d4d
Change-Id: I335495b8672c7b4d187400fa8b0abe511b110688
2024-06-01 18:25:41 +02:00
Christian Soffke
3d7d1d4d5b plugins: properties: show track info for whole playlist
Track Info can now be displayed for the set of
all tracks contained in a  playlist. This lets
you calculate a playlist's length, for example,
even if it is not currently playing.

This functionality can be accessed from the
existing "Properties" screen for a selected
playlist file. A line has been added at the
very bottom to show Track Info.

Change-Id: I311532b7cfa9e29d46c0cd5623ba4c06c1dd5b5f
2024-06-01 11:55:36 -04:00
Christian Soffke
ef1e7d8896 bookmarks: show 'Create Bookmark' for unsaved playlists
Instead of hiding this option when the current playlist
is unsaved, we can offer to save the playlist, if
necessary, before attempting to create a bookmark.

This simplifies the workflow by getting rid of steps 1
and 2 that a user currently needs to perform:

1) Open context menu, check if "Create Bookmark" option
   is available
2) If not: Re-open context menu. Select Current Playlist
   -> Save Current Playlist
3) Re-open context menu. Select Bookmarks
   -> Create Bookmark

Change-Id: Ia9fb5f9db95e2fcbde03470a3c3745824424a339
2024-06-01 11:42:47 -04:00
William Wilgus
d1aeb7db9d [Coverity] playback.c audio_auto_change_frequency check for valid id3
Change-Id: Ia213134c690f4c1de16afba38eea740825558280
2024-05-31 23:02:28 -04:00
Solomon Peachy
eaeaac598d FS#13431: Provide a way to view the rockbox license in the menus
This adds a "Legal Notices" entry under the System menu that will
launch the text viewer plugin to display:

1) COPYING.txt   (containing the GPLv2 overall license for Rockbox)
2) LICENSES.txt  (containing all other relevant licenses and notices)

Change-Id: Id41ae491b6f9a6ec5663090246d0f6b661cddea5
2024-05-30 21:22:01 -04:00
Solomon Peachy
f0c1cf1eef Update US English "Translation"
Change-Id: I60b26a8d7d6ab02d017070581b9d28d4e4793293
2024-05-30 20:01:41 -04:00
Solomon Peachy
09e01c89a7 FS#13430: Updated German translation (Wilfried Winkler)
Change-Id: I0afd0468cafd22dea291eff044b86fd5b637468e
2024-05-30 19:49:16 -04:00
Franklin Wei
e72bae7c3f [Superdom] Bugfix
Prevents the plugin from locking up on the highest difficulty level

Change-Id: I07ab77e825935948b2d15ca6d36ad24b2716ac1b
2024-05-26 11:51:30 -04:00
Solomon Peachy
8b8c0c7c2a checkwps: Validate translated strings (%Sx)
In verbose mode it will log valid the strings found, otherwise it will
only complain when we encounter a missing string.

Unfortunately a missing string is not inherently a problem, due to
conditional expressions.  So all we can do is complain in checkwps
or if wps debugging is turned on.

Meanwhile, this is the first step in actually enumerating the translated
strings used by themes.

Change-Id: Ia93b333085e825d5b085c4d372ad8e13aa3e3ba1
2024-05-24 21:37:54 -04:00
Solomon Peachy
ac20f8a73c lang: Delete 12 unused strings
Change-Id: I7b1d98a0eb4c58cdf5c1819be6e2125e9261246b
2024-05-24 12:11:00 -04:00
Solomon Peachy
2dc6947a2d lang: Get rid of *unused* LANG_EQUALIZER_EDIT_MODE and all deprecated strings
The documentation says to not delete things but that dates back to when
we were a lot laxer in what we would accept at runtime.

I want to make things even stricter, and add a checksum or hash of
the string IDs, so that *any* change to the base language invalidates
translation and voice binaries at runtime.

Change-Id: I94a71781f839bec804dc04dd99660e5e866577bf
2024-05-22 08:52:17 -04:00
Solomon Peachy
62da4ad84b voice: Correct all uses of spashf() with an ID2P() argument
...Instead voice the string and all of the arguments manually.

Change-Id: Ic030b2f00c90cb372ff5b781a8de4bcb19375d5d
2024-05-19 21:05:43 -04:00
Solomon Peachy
dfe409c993 voice: Revert one of the changes made in b6053c4d54
We already voiced the full text displayed in that splash message.

Change-Id: I40cf3de9124f9614b139fae41c47ef2ce7fb227a
2024-05-19 21:05:43 -04:00
Solomon Peachy
76de880ee0 lang: Replace "DAC's" with "DAC" in English source.
Fix up the translations afterwards

Change-Id: I58594aa8022e071b2f1ccc071562de2512962b31
2024-05-17 08:31:09 -04:00
Solomon Peachy
b9755a3085 A handful of language cleanups
Change-Id: If828e6c6d13ec8ef15d829d6c8fb950ad3af3f75
2024-05-16 21:51:14 -04:00
Solomon Peachy
eb71060e96 lang: Run the master English language file through a cleanup pass
Do a cleanup pass on english-US too.

Change-Id: I1b13d1c05585ef7258378d8a89bfe01cba0d7b36
2024-05-16 21:16:25 -04:00
William Wilgus
9d3b012ac0 [BugFix, Asan] skin_tokens.c get_dir path name underrun
preventing multiple slashes reads prior to the buffer by 1 byte
when only a single slash exists

Change-Id: I1ce1dc8d754bf1c8798dc49f396c9142e4a6e49c
2024-05-13 17:19:45 -04:00
Christian Soffke
556b466f05 shortcuts: allow empty name for sleep shortcuts with default duration
missed this in f631bfe

Change-Id: Ifdb3ffc04b49133b770850b98399605c04742f6e
2024-05-13 21:26:30 +02:00
Solomon Peachy
b94e5d77fe Update English_US "translation"
Change-Id: I94bcbb86ab8421aee248553dc38fd13bb6573cf7
2024-05-13 14:21:26 -04:00
Solomon Peachy
08c35a2299 FS#13422: Update Polish Translation (Adam Rak)
Change-Id: Ida6a7315645102a27732dc66809dc9ee922f9b46
2024-05-13 14:21:26 -04:00
Christian Soffke
fdba79cd77 shortcuts: refactor sleeptimer / talk_timedate
move some functions around, with no effect on behavior

Change-Id: I4638a28f5ff2a851534a3dd696ea7e763029cb2f
2024-05-13 18:18:45 +02:00
Christian Soffke
8eeef333a1 shortcuts: add 'reboot' type
Enabled by d55dcef

Change-Id: I689e289feb4715aab603bae4a6855cf5e227562b
2024-05-13 18:06:55 +02:00
Christian Soffke
f631bfe5b4 shortcuts: sleep timer: allow omitting number of minutes
'sleep' can now appear in the data field of a 'time' shortcut
without being followed by a number, allowing you to stop a
running timer, or to start a new one using the default sleep
timer duration (the duration setting can already be added to
the Shortcuts menu as well).

Also see here:
https://forums.rockbox.org/index.php/topic,54312.msg250940.html

Change-Id: I9d0e62ef1b6187c35133067349729a4d94273c7a
2024-05-13 18:06:55 +02:00
Christian Soffke
b8b4fdd999 shortcuts: voice fixes
- current time wasn't spoken
- 'sleep' shortcuts were voiced as "Time & Date"
- selected item wasn't spoken when returning
from Delete confirmation dialog

Change-Id: I3a3e8b7782323a8c02ae07302ecdfdd742e60908
2024-05-13 18:06:55 +02:00
Christian Soffke
51a5123f5e shortcuts: 'browse' type: fix ignored name field
see https://forums.rockbox.org/index.php/topic,54619.0.html

Change-Id: I9615ec9ee2e058a67199db96e16cd7008daf0891
2024-05-13 18:06:55 +02:00
Solomon Peachy
b6053c4d54 voice: Fix a handful of non-voiced splash messages
Change-Id: I561c756c542fa24824a881b5632c955a920b8471
2024-05-13 12:02:17 -04:00
Aidan MacDonald
cc45e4d88d Really fix yellow this time
Change-Id: I73a39f823ca5d20f8a562ddb36ba4f8c7137b2db
2024-05-12 15:21:56 +01:00
Aidan MacDonald
2061448341 Fix yellow in 5f377c2613
Change-Id: Ib593e7afea52bc768242aa93d304f38fa1aea054
2024-05-12 15:05:21 +01:00
Dana Conrad
5f377c2613 Eros Q Native: Add Stereo SW behavior setting
Hopefully this should cover our bases so we can change
the behavior of the stereo switch to keep line out working
when they change the hardware on us!

Change-Id: Ic36bcb3778d5681a5f3f158c689df9c1420c1d7e
2024-05-12 09:42:02 -04:00
William Wilgus
4128a1fe48 [Bugfix/Feature] OpenPlugin and default plugins
selecting files to run is nice and all but you might not like the plugin
you can edit it OpenPlugin Viewer Plugin but instead pop it when you add
a file to reduce suprises

shortcut viewer is not ready for this so exclude it for now

Change-Id: I950599d87f47d42e8c2d59695f6583d497b217f0
adds: default plugin (if any) is selected in the open with dialog
2024-05-09 16:13:24 -04:00
Solomon Peachy
1957237a46 Fix red in 8c86fb6da0 (ipod5g only)
Change-Id: Idf6a488728b0187779bc4a9232c7e7e9400afeec
2024-05-08 22:03:59 -04:00
Solomon Peachy
8c86fb6da0 arm: Use -masm-syntax-unified when compiling with gcc8 or newer
Annoyingly, this makes all of the '.S' files we compile get treated as
divided syntax, so we need to make the syntax in them explicit.

Change-Id: I56a3916b7b24c84a1214a5d6bc4ed4d651f002cf
2024-05-08 21:45:42 -04:00
Solomon Peachy
0a89d1d4df Fix a typo in English
Change-Id: Ia0b84338cecbc744050336ac49074ec1a559f51a
2024-05-08 10:42:00 -04:00
William Wilgus
3348d84206 open_plugins Fix Yellow
Change-Id: I036bd57c7e55f7059f858e9f0929500c2094e579
2024-05-06 18:00:49 -04:00
William Wilgus
ee840709d3 [Feature] Open plugins now recognizes known filetypes and can run them
now you can run your lua files without having to add the viewer to
the shortcut or if you want a bmp file to be displayed when you start
the device that can be done as well

Change-Id: Ia56b566789623a2ca78d9e4583086db6e2cd689b
2024-05-06 17:27:25 -04:00
William Wilgus
30482bd908 [BugFix] Radio make sure resume frequency is in range
out of range frequencies hang the device (clip zip, others?)

Change-Id: I0d3af83a05479f70a958168f57c8cc305195b06b
2024-05-06 10:28:27 -04:00
William Wilgus
f55483a966 lua --remove files moved to include directory
Change-Id: Ic9256cb28126dba89b1458109c717f846f5b6d38
2024-05-05 14:11:47 -04:00
William Wilgus
d7c541742f Allow first level folders in plugin menu
add sorting directories as files
move picross files to a hidden folder

use directory for lua_scripts, sgt_puzzles

make plugin browser able to handle 1st level directories

Change-Id: I30852d71dc992c378d5790756e94f06f5a2e9bef
2024-05-05 14:01:43 -04:00
William Wilgus
efcea66280 Revert "readdir_r use in tagcache.check_dir, ft_load"
This reverts commit 0c737d3b2e.

Reason for revert: Not really a concern as open_stream returns an independent buffer since g#566

Change-Id: Idbd2f4a7cc2ea6362b7714629469eeb7b3d19b3b
2024-05-02 13:38:32 -04:00