RISC OS Open
Safeguarding the past, present and future of RISC OS for everyone
ROOL
Home | News | Downloads | Bugs | Bounties | Forum | Documents | Photos | Contact us
Account
Forums → Bugs →

09/04/17 ROM breaks Screen config/Netsurf

Subscribe to 09/04/17 ROM breaks Screen config/Netsurf 13 posts, 8 voices

 
Apr 9, 2017 2:53pm
Avatar Frederick Bambrough (1372) 799 posts

Open a Netsurf window. Open Configuration – Screen. Move Netsurf window and enjoy the fireworks.

Haven’t checked any other app windows yet.

Edit: Seems to be just Netsurf. Tried Fireworkz, OPro, Pipedream, StrongEd, Zap & LaBella all of which seem OK.

 
Apr 9, 2017 3:43pm
Avatar Doug Webb (190) 1074 posts

Frederick

What version of Netsurf is that with and also what versions of support modules that it requires have you installed.

Also I assume you have updated your disc image as well?

 
Apr 9, 2017 4:00pm
Avatar Frederick Bambrough (1372) 799 posts

Disc image is today’s. Netsurf is 3.7 (Dev C1 #4059). Support modules; AcornURI 0.26, Iconv 0.12, SharedUnixLibrary 1.14, Tinct 0.14

What’s happening is that after initial breakup of Netsurf’s window content any window passing over the Netsurf one leaves a trail but only after opening the screen config.

 
Apr 9, 2017 4:05pm
Avatar Chris Gransden (337) 1105 posts

On a Titanium after running Configuration→Screen the NetSurf window goes ’Fireworks. Changing the mode to the current mode gets things back to normal or pressing F12, Return the desktop screen mode changes to 640×480 in 2 colours.
On a Raspberry Pi 3 a higher resolution is selected. Goes from 2048×1152 to 2560×1440. Colours stay the same. Netsurf remains unaffected.

 
Apr 9, 2017 4:12pm
Avatar Frederick Bambrough (1372) 799 posts

Screen resolution here is 1024×768, the max & native for this Dell 1505FP. Beagleboard -xM.

 
Apr 9, 2017 4:30pm
Avatar Steve Pampling (1551) 7461 posts

Disc image is today’s. Netsurf is 3.7 (Dev C1 #4059). Support modules; AcornURI 0.26, Iconv 0.12, SharedUnixLibrary 1.14, Tinct 0.14

Try without the use of Tinct1 by selecting OS based dither in the netsurf image handling config.

1 May be irrelevant but I do recall strange things happening with Tinct use on releases a few years ago.

 
Apr 9, 2017 4:39pm
Avatar Jeffrey Lee (213) 6028 posts

The Wimp probably needs fixing so that it will only refresh currentmode on Service_ModeFileChange if the desktop hasn’t been entered yet. currentmode is directly exposed by Wimp_ReadSysInfo 1 (it’s supposed to be the current Wimp mode), so once you’re in the desktop it should only really change on when an actual (Wimp) mode change occurs.

 
Apr 9, 2017 4:48pm
Avatar Frederick Bambrough (1372) 799 posts

Try without the use of Tinct1 by selecting OS based dither in the netsurf image handling config.

Already selected.

 
Apr 11, 2017 1:08pm
Avatar Sprow (202) 1048 posts

The Wimp probably needs fixing so that it will only refresh currentmode on Service_ModeFileChange if the desktop hasn’t been entered yet.

Wouldn’t it be simpler to only refresh it when the task count changes from 0 to 1? I assume (though can’t actually see the line) it’s a call to Wimp_Initialise that triggers the initial mode change, in response to the *Desktop command starting a Wimp task.

Then there’d be no need to bother reading it when the module initialises, nor on Service_Reset, nor on Service_ModeFileChange.

But since I can’t see where the initial mode changes by just staring at it, I’ve no doubt missed something subtle and would need to stick a breakpoint in to see what’s going on.

 
Apr 11, 2017 6:54pm
Avatar Jeffrey Lee (213) 6028 posts

But since I can’t see where the initial mode changes by just staring at it, I’ve no doubt missed something subtle and would need to stick a breakpoint in to see what’s going on.

Use the force, Luke! grep, Sprow!

The main place to worry about is Wimp_SetMode / *WimpMode. If the wimp isn’t active yet then it won’t actually trigger a mode change; all it will do is update currentmode, thereby selecting the screen mode that should be used when the Wimp does get entered. The boot sequence relies on this behaviour – the Configure.Monitor file is located in PreDesk.

 
Apr 12, 2017 3:29pm
Avatar Frederick Bambrough (1372) 799 posts

Just to confirm, today’s ROM (11/12 Apr) has resolved the problem on this BB -xM.

 
Apr 12, 2017 4:52pm
Avatar John Williams (567) 768 posts

Just to confirm, today’s ROM (11/12 Apr) has resolved the problem on this BB -xM.

I would like to congratulate whoever fixed this, and ask that the revisions page actually says who is responsible for updates – not to criticise failures, but to congratulate fixes for bugs and so forth.

Too often it says just ROOL, and the individual is anonymised.

I want to be able to say “Thank you, whoever!”

 
Apr 13, 2017 11:01am
Avatar Chris Evans (457) 1613 posts

Too often it says just ROOL, and the individual is anonymised. I want to be able to say “Thank you, whoever!”

I agree with your comment.
If you dig deep enough into CVS I’ve often found a view of the sources that shows who wrote what, line by line with the authors initials but I often don’t seem to be able to find it.
I’ve just tracked down another view of the relevant sources here. That lists Rob Sprowson as the author of the above fix, though it did require a bit of digging included changing the view to sort by date.

Reply

To post replies, please first log in.

Forums → Bugs →

Search forums

Social

Follow us on and

ROOL Store

Buy RISC OS Open merchandise here, including SD cards for Raspberry Pi and more.

Donate! Why?

Help ROOL make things happen – please consider donating!

RISC OS IPR

RISC OS is an Open Source operating system owned by RISC OS Developments Ltd and licensed primarily under the Apache 2.0 license.

Description

Bug discussions that aren’t covered by the bugs database.

Voices

  • Frederick Bambrough (1372)
  • Doug Webb (190)
  • Chris Gransden (337)
  • Steve Pampling (1551)
  • Jeffrey Lee (213)
  • Sprow (202)
  • John Williams (567)
  • Chris Evans (457)

Options

  • Forums
  • Login
Site design © RISC OS Open Limited 2018 except where indicated
The RISC OS Open Beast theme is based on Beast's default layout

Valid XHTML 1.0  |  Valid CSS

Powered by Beast © 2006 Josh Goebel and Rick Olson
This site runs on Rails

Hosted by Arachsys