Age | Commit message (Collapse) | Author |
|
This fixes a bug reported on IRC:
1. Set 'bookmark on stop' to 'ask'
2. Play a track from the file browser
3. Stop playback, yes/no screen briefly flashes then disappears.
The screen still handles input correctly but the prompt will not
be displayed on the LCD. Long story short, get_action() can cause
the skin engine to do a full redraw which cleared the screen and
erased the prompt.
There is a special event, GUI_EVENT_NEED_UI_UPDATE, which the list
code uses to avoid precisely this problem.
Hooking up a handler for this event and redrawing the yes/no prompt
fixes the bug.
Change-Id: If3c26655540c13b488b3a11bd46e1d60d4111469
|
|
These were deprecated by 1eb17dc9f4 back in 2013 just prior to the
3.13 release. We've had 3.14 and 3.15 since then and I guess nobody
will complain if we finally remove them (fingers crossed...)
Also fix a line which got missed when converting hardcoded numbers
to EQ_NUM_BANDS in that commit.
Change-Id: I79dd71f1f2810bf19bc135999868fafc5fee6e04
|
|
add a helper function to allow plugins to parse the parameter string
passed into plugins
support included for bool, char, string and numbers+decimals
Change-Id: I39f35c8bd3c21b83097a538c19f46d362c468fa4
|
|
The EQ enabled setting needs to call into the DSP core when it's
changed but this was only hooked into the EQ menu. The quickscreen
uses option_select_next_val() so those DSP hooks were not getting
called because the setting did not have any callback defined.
It seems this bug was introduced by 362f7a3220 removing the blanket
call to settings_apply(). Other settings might be affected too.
Change-Id: Idcce5809c525fa619e6bdf1be50c98a5df103fec
|
|
Change-Id: I4cded8733f8ae4d26b013a944ee5e884ed1363d9
|
|
share all the core icons with plugins
these are all small mono icons like usb plug icon or
play, fast forward, rewind icons
--include the icon_helper function
Change-Id: I385028815a4dd368515f491a9e19dee3d500252d
|
|
- Enable repeat for prev/next keys
- Provide a way to access the context menu using buttons
- Update the manual
Change-Id: I53a7474cc64f915851175aea0695229c581fd8a6
|
|
This fixes those annoying jumps that happen when you hit the end of
a list while scrolling.
Change-Id: I2e41111f9415dce1692b52a2600e7ce77c8f0291
|
|
Change-Id: I3b55233f4d50f5b8ae38f9f337ebdee6f93ad44e
|
|
We want to load builtin filetypes and other config files even if
there is a problem loading viewers.config. A lot of menus expect
the builtin types to be there and don't work if they're missing.
Change-Id: Ie39c8b9ef184fe0d638bacbe18e5f2d22900bd81
|
|
Has been disabled since HW codec support was dropped, but works fine. Tested
on Surfans F20.
Change-Id: I97f06a009048e8135ca21118467c491e7077fa4f
|
|
talk_value_decimal() replaced talk_value()
Change-Id: Ibb7c738d6f3b399fab2e771d8ab85967e3af96d1
|
|
talk_value is just talk_value_decimal with 0 decimals
lets add the extended function instead
static inline int talk_val(long n, int unit, bool enqueue)
{
#define NODECIMALS 0
return rb->talk_value_decimal(n, unit, NODECIMALS, enqueue);
}
Change-Id: Iaba3d2f95785f2e1855e294ccf099a977bb6cb20
|
|
optimize +
add some error checking of inputO
Change-Id: I88c476f5fa66847adc0b361df1a0dd4818dbfdc6
|
|
Only Archos players ever used this feature and it isn't implemented
on any current targets.
Change-Id: I39c8455480a3fe33cd25b667e8d4d6ff3a092d3e
|
|
read errors are negative buf_sz was a unsignbed int
Change-Id: I45ba67e09ce54ff09411248340ba2c9c62c57583
|
|
Change-Id: I14ece34e5ad0ffe00bbb4f7de37b1dadad3b92b4
|
|
It was barely even a stub, and hasn't seen any development since 2009
Change-Id: I0aa15d9a7b90ae8c771924d9f401380d4cc0fab9
|
|
It has only a couple GB of onboard flash storage with no storage
driver in Rockbox. It hasn't seen any meaningful development since
its initial commit, it's not on the build farm, and the bootloader
build is broken by other refactoring.
Change-Id: Idd9e3c46fac9b96f416ce444182f97a50770e747
|
|
Expands upon c067b344e8. The bug mentioned in that commit affects
touchscreens as well as touchpads, plus touchscreens were subject
to a few other issues because 'last->tick' was not always set.
Change-Id: I27eaa045ebf09555259fddbdf3a79828a55ae31f
|
|
Change-Id: I0293371c58f1ca2d148b3b1e1f780cf76f312ef9
|
|
Change-Id: I253e19695709b4184ceb4a0748300f7c68437c1c
|
|
sbuf goes out of scope of the case statement
Change-Id: Ice0553b496bfbfa5bc243dddf5bad90f3baaba51
|
|
Change-Id: I1e2acceb707d85481eace3203b1d3d3ec8beb7d1
|
|
Change-Id: I58941cea061f41478d7677beb9ff42e28b2455a6
|
|
Some lists have tall items that span more than one line of text,
eg. the bookmark menu or ID3 tag menu. The touchscreen code didn't
handle these menus correctly and touching on the lower part of a
list item could select "between" two items, leading to incorrect
rendering and behavior due to callers relying on the selected item
being properly aligned to the selection size. Fix this by ensuring
the touch code only generates properly aligned selections.
Change-Id: I73945bb0947590517a005754bd447639e22812e2
|
|
Changing the touchscreen mode in the settings menu required a reboot
to fully apply because this code was trying to reapply old settings.
This is a hack. IMHO to fix it properly the behavior of touchscreen
mode needs to be changed so code that needs a specific mode doesn't
need to twiddle global state, but that's a bigger undertaking.
Change-Id: I55a46dbed721273ad78cb9df81fe60940665962b
|
|
Seems to fix the unintuitive / inaccurate behavior of this screen.
Rejigger the generic touchscreen keymap because the 3x3 buttons are
no longer used. Also map physical keys for yes/no on the Q1.
Change-Id: I2ed633896b87030fa654d47d9d02929b849495da
|
|
Change-Id: Ifab0e607cb71afbe26b9e508fe1db3a8730bad4b
|
|
Change-Id: Id31d3d8b571540394373a61ed213380c95a0eee2
|
|
Share a bit of code in order to free up iRam for the iAudio M5
Change-Id: Ief05d2614a203df17821ed32b04f5c75a53ca08b
|
|
re: coverity
write_write_order: In long(*s++) + *s++,
s is written in *s++ and written in long(*s++)
but the order in which the side effects take place is undefined because
there is no intervening sequence point.
Change-Id: I2911c240f3e85fcfbf77297e8579e02e217c5af5
|
|
Change-Id: I21035cda0617e74283e4b30557783d297d8dfcd0
|
|
Change-Id: I88fe17466b606a1f60a4c823214edd9766deb260
|
|
Change-Id: Ice86f060974c51bbaf051ed8c5a369ce80ecfe15
|
|
Forgot to commit this back around the time I did the manual
Change-Id: I8fafeec5105b4bee3380779f4cc86d3e73459cf2
|
|
Change-Id: I9c539035544d285da1941d2c1954cf2aa4d00087
|
|
Change-Id: I59d88609ec3f20ec9f482a777396cd72be23a35b
|
|
Change-Id: I000c78e913ab80afa9b82cae937afb5255831b79
|
|
Change-Id: Ibf754554f685d13ac00a49694fcecee459ff838f
|
|
...Log the variable containing the actual error
Change-Id: If22e6b684f2e213bc45c8ca95295936ba8e09895
|
|
Change-Id: I6df903399361f1df7ca0f766fff4062a984e2ba8
|
|
Change-Id: I7085cb34d2258aa5fc7e04fc0e73d6f733810038
|
|
Surprise surprise: the compiler generates shorter, branch-free code
if we don't try to be clever with signed/unsigned casting tricks.
Change-Id: I93d2020b6127e7d43feee394b06a52aaeddf3b79
|
|
Change-Id: I14f3d83a8089d33f4e900a1d5f965e67082a07ea
|
|
One of these was an unintentional fallthrough -- if you said no to
'quit without saving' it would save and quit.
Change-Id: I33828b58962a05adab65ac5f71dd0e2b1e07001b
|
|
Change-Id: I1ca5b1027ec30cbf61093bab35b980196ed14e6b
|
|
Change-Id: I499bcd6d2d6fd5dd92c184cc1ec014e62f0441fa
|
|
Change-Id: Id45d9ac3b88571cb3927ca29314f997d5cbfd598
|
|
I'm not sure this would happen but I suspect its possible to get malformed
data back from lineinfo as this is how the error system tracks code flow
and this is a hacked add-on to allow RLE (run length encoding) of
said code flow data
Change-Id: I4b914c632935f8b1a9fa960212c58e810b2db738
|