12:26 p.m.
I have previously patched netsurf and a couple of test programs to automatically select
the appropriate nsfb backend using getenv() if it is not provided as an arg, but it may be
useful to have this as the default behavior.
the following env variables are set:
Wayland -> WAYLAND_DISPLAY
xcb -> DISPLAY
linux framebuffer -> FRAMEBUFFER (and possibly isatty()?)
vnc -> check port 5900?
able -> no idea
For the ones I'm familiar with the order should be wayland,xcb,linuxfb,sdl but I am
less familiar with able and vnc. I assume Able should fall somewhere around the linuxfb
and vnc should either be first (by checking port 5900) or required as an arg.
10:46 a.m.
On 07/09/2013 18:26, Brad Conroy wrote:
I have previously patched netsurf and a couple of test programs to
automatically select the appropriate nsfb backend using getenv() if it is not provided as
an arg, but it may be useful to have this as the default behavior.
the following env variables are set:
Wayland -> WAYLAND_DISPLAY
xcb -> DISPLAY
linux framebuffer -> FRAMEBUFFER (and possibly isatty()?)
vnc -> check port 5900?
able -> no idea
For the ones I'm familiar with the order should be wayland,xcb,linuxfb,sdl but I am
less familiar with able and vnc.
I assume Able should fall somewhere around the linuxfb and vnc should
either be
first (by checking port 5900)
or required as an arg.
For SDL you'll want to check those, even though they aren't mandatory:
http://sdl.beuc.net/sdl.wiki/SDL_envvars
François.
8:01 p.m.
I just wanted to follow up on this before I start making any changes.
It looks like the appropriate place to implement auto-backend select is:
src/libnsfb.c
nsfb_t*
nsfb_new(const enum nsfb_type_e surface_type)
{
nsfb_t *newfb;
newfb = calloc(1, sizeof(nsfb_t));
if (newfb == NULL)
- return NULL;
+{
+ /*autoselect code here: if (getenv("WAYLAND_DISPLAY")) ... */
+}else{
It looks like modes that are not supported still get checked,
but could be skipped if libnsfb.h had some #ifdefs for:
enum nsfb_type_e {
NSFB_SURFACE_NONE = 0, /**< No surface */
NSFB_SURFACE_RAM, /**< RAM surface */
#ifdef NSFB_SDL_AVAILABLE
NSFB_SURFACE_SDL, /**< SDL surface */
#endif
#ifdef NSFB_LINUX_AVAILABLE
NSFB_SURFACE_LINUX, /**< Linux framebuffer surface */
#endif
#ifdef NSFB_VNC_AVAILABLE
NSFB_SURFACE_VNC, /**< VNC surface */
#endif
#ifdef NSFB_ABLE_AVAILABLE
NSFB_SURFACE_ABLE, /**< ABLE framebuffer surface */
#endif
#ifdef NSFB_XCB_AVAILABLE /* naming inconsistent */
NSFB_SURFACE_X, /**< X windows surface */
#endif
#ifdef NSFB_WLD_AVAILABLE
NSFB_SURFACE_WL /**< Wayland surface */
#endif
/*put a dummy here so it is always last? and can be used in loops?*/
/* it looks like default order is the reverse of this */
};
Note: these would need to be define in a nsfb-config.h
BTW, the default width and height of 800x600 could be problematic on small devices
Is there a reason it cannot be clamped to the screen resolution on a per surface basis?
3543
days inactive
3557
days old
netsurf-dev@netsurf-browser.org
3 comments
3 participants
participants (3)
-
Brad Conroy -
François Revol -
Rob Kendrick