Bug in Fat32FS Filer Taskbar
Stefan Fröhling (7826) 167 posts |
I experience this layout bug that the space reserved for the name is too slim for the name to fit into. Screenshot of Name display bug Who can fix it? |
Rick Murray (539) 13840 posts |
I think the guess isn’t good. #ifdef SHOW_DRIVE_NAME_ON_ICON_BAR regs.r [1] = (int) text; _kernel_swi (Font_StringBBox, ®s, ®s); x = regs.r [3] - regs.r [1]; // Max X - Min X x = x / 360; // Divide by 360 is a guess if (x < 68) x = 68; In source → module → c → wimp. There’s a lot of wrongness there. StringBBox says “This call cannot, due to rounding and anti-aliasing, be used to find size on screen”. ;) And while the default conversion is 400 millipoints to an OS unit (there’s the call Wouldn’t it be better to replace all of that with Wimp_TextOp, 1? |
Stefan Fröhling (7826) 167 posts |
Yes! Wimp_TextOp, 1 should be used here. |
Stefan Fröhling (7826) 167 posts |
@Rick I tried to find it in GitLap RISC OS sources but failed. |
David Pitt (9872) 363 posts |
It is time to get the latest version of Fat32FS 1.63 09Jan24. See changelog in the ReadMe. The code is in |
Stefan Fröhling (7826) 167 posts |
@David Great ! Thanks! |
Stefan Fröhling (7826) 167 posts |
One good news and one bad news: |
David Pitt (9872) 363 posts |
Seems all OK here on an RPi4 and Titanium with recent OS5.29s. Does |
Stefan Fröhling (7826) 167 posts |
I found out more now: So something goes wrong with initialzing the menu when the drive is opened already. In version 1.56 it didn’t happen.
What does it mean? WHat can I do get more debugging infos? here list of modules: |
Andrew McCarthy (3688) 605 posts |
Here’s a link to the RISC OS Debugging guide.
Another link that describe the “where” command and others. Edit: the following description of the where command fits my understanding. Is there anyone comfortable enough with Textile able to update the wiki entry? |
David Pitt (9872) 363 posts |
I still cannot reproduce any error.
|
Colin Ferris (399) 1814 posts |
As well as You could add *SHOWREGS |
Stefan Fröhling (7826) 167 posts |
Well there must be a difference in what is done when initialize the menu when is “empty”(no drive initialized) and when a drive is moouted. By the way why the drive is not automatically initialized when plugged in? Windows does that and I think i makes sense as anyhow the pluggin in is detected as otherwise the USB stick doesn’t show… |
Stefan Fröhling (7826) 167 posts |
PS: About the debugging stuff I will look later today. Now no time for that. |
Stuart Painting (5389) 714 posts |
AIUI, the drive is initialised, but only SCSIFS is aware of its existence to begin with. Fat32Fs only gets interested in the drive when you left-click on the icon. You can choose to bypass Fat32Fs by right-clicking on the icon: this is most likely the reason why Fat32Fs isn’t interested in device insertions. |
Stefan Fröhling (7826) 167 posts |
Where &FC348E2C : is at offset adress &00001C54 of module SCSIFiler |
Stefan Fröhling (7826) 167 posts |
Register dump (stored at &2003D870) is: |
Stefan Fröhling (7826) 167 posts |
Most people will not discover this bug as it only happens when the task bar menu is opened. And then likely not to happen with all drives or device names. |
Stefan Fröhling (7826) 167 posts |
Can you send me older version of FAt32FS like 1.59 and 1.60 (or all) ? The version that didn’t crash is 1.56. |
Stefan Fröhling (7826) 167 posts |
@David |
Raik (463) 2061 posts |
A little bit offtopic… Toni has send an FAT32FS bugfix in 02/2020 especially for the Ti SATA to Jeff. Looks like is not inside the latest version and older. So for me the latest version is not usable. |
Raik (463) 2061 posts |
I take everything back and claim the opposite. |