[compositor] idle-time configuration inside weston.ini

[Marc Chalain]

You have right, it's the smallest of my patches, and the worst.

[PATCH] [compositor]: add "idle-time" entry support in "core"
 section of weston.ini

---
 src/compositor.c |    2 ++
 1 file changed, 2 insertions(+)

diff --git a/src/compositor.c b/src/compositor.c
index 099600d..2002812 100644
--- a/src/compositor.c
+++ b/src/compositor.c
@@ -3277,6 +3277,8 @@ int main(int argc, char *argv[])
     section = weston_config_get_section(config, "core", NULL, NULL);
     weston_config_section_get_string(section, "modules",
                      &modules, "desktop-shell.so");
+    weston_config_section_get_int(section, "idle-time",
+                     &idle_time, idle_time);

     backend_init = load_module(backend, "backend_init");
     if (!backend_init)
-- 
1.7.9.5



2013/6/10 Pekka Paalanen <ppaalanen at gmail.com>

> On Fri, 7 Jun 2013 12:39:21 +0200
> Marc Chalain <marc.chalain at gmail.com> wrote:
>
> > this patch use the "idle-time" entry inside the "core" section of
> > weston.ini to set the named value.
>
> Hi Marc,
>
> does this actually work?
>
> I think the backend init calls weston_compositor_init(), where your
> code then loads the idle-time from config, and then later in main()
> ec->idle_time is unconditionally set. To me it looks like the config
> setting gets always overwritten.
>
> Also patch the man pages for this change, please.
>
>
> Thanks,
> pq
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.freedesktop.org/archives/wayland-devel/attachments/20130611/05a87455/attachment.html>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-compositor-add-idle-time-entry-support-in-core-secti.patch
Type: application/octet-stream
Size: 825 bytes
Desc: not available
URL: <http://lists.freedesktop.org/archives/wayland-devel/attachments/20130611/05a87455/attachment.obj>