1. 15 Mar, 2016 2 commits
  2. 04 Mar, 2016 15 commits
  3. 03 Feb, 2016 1 commit
  4. 14 Dec, 2015 3 commits
  5. 27 Nov, 2015 4 commits
  6. 29 Oct, 2015 1 commit
  7. 01 Oct, 2015 5 commits
  8. 23 Sep, 2015 1 commit
  9. 26 Aug, 2015 1 commit
  10. 21 Aug, 2015 3 commits
    • eucan's avatar
      ivi-shell: remove redundant wl_list_empty/init calls · cf34dc23
      eucan authored
      Checking wl_list_empty() on a link offers no information: if it returns
      true, wl_list_remove() is safe to do. If it returns false, you still do
      not know if wl_list_remove() is safe; the link could be part of a list,
      or the link could be "uninitialized" (e.g. just wl_list_remove()'d).
      (From Pekka Paalanen's comment at
      http://lists.freedesktop.org/archives/wayland-devel/2015-August/023987.html).
      
      Calling wl_list_init just before wl_list_insert is redundant.  Because
      the links of the list are not read before it is overwritten by
      wl_list_insert.
      Signed-off-by: eucan's avatarEmre Ucan <eucan@de.adit-jv.com>
      [Pekka: line-wrapped commit message]
      Reviewed-by: Pekka Paalanen's avatarPekka Paalanen <pekka.paalanen@collabora.co.uk>
      cf34dc23
    • eucan's avatar
      ivi-shell: clear order.layer_list before reordering it · 174257bd
      eucan authored
      It is only possible to remove a layer from the order.layer_list of a
      screen, when ivi_layout_screen_set_render_order is called with an empty
      array.  Therefore, list of layers are cumulated if the API is called
      many times with different list of layers.
      
      Change how the flags are set:
          - Introduce the dirty parameter for triggering the render order change.
          - IVI_NOTIFICATION_REMOVE/ADD flags are set only at commit_screen_list.
      
      Checking wl_list_empty() on a link offers no information: if it returns
      true, wl_list_remove() is safe to do. If it returns false, you still do
      not know if wl_list_remove() is safe; the link could be part of a list,
      or the link could be "uninitialized" (e.g. just wl_list_remove()'d).
      (From Pekka Paalanen's comment at
      http://lists.freedesktop.org/archives/wayland-devel/2015-August/023987.html).
      
      Calling wl_list_init just before wl_list_insert is redundant.  Because
      the links of the list are not read before it is overwritten by
      wl_list_insert.
      
      Use assert to control if the ivilayer->order.surface_list is empty.
      Signed-off-by: eucan's avatarEmre Ucan <eucan@de.adit-jv.com>
      [Pekka: line-wrapped commit message]
      Reviewed-by: Pekka Paalanen's avatarPekka Paalanen <pekka.paalanen@collabora.co.uk>
      174257bd
    • eucan's avatar
      ivi-shell: clear order.surface_list before reordering it · 38fcf381
      eucan authored
      It is only possible to remove a surface from the order.surface_list of a
      layer, when ivi_layout_layer_set_render_order is called with an empty
      array.  Therefore, list of surfaces are cumulated if the API is called
      many times with different list of surfaces.
      
      Change how the flags are set:
      - Introduce the dirty parameter for triggering the render order change.
      - IVI_NOTIFICATION_REMOVE/ADD flags are set only at commit_layer_list.
      
      Checking wl_list_empty() on a link offers no information: if it returns
      true, wl_list_remove() is safe to do. If it returns false, you still do
      not know if wl_list_remove() is safe; the link could be part of a list,
      or the link could be "uninitialized" (e.g. just wl_list_remove()'d).
      (From Pekka Paalanen's comment at
      http://lists.freedesktop.org/archives/wayland-devel/2015-August/023987.html).
      
      Calling wl_list_init just before wl_list_insert is redundant, because
      the links of the list are not read before it is overwritten by
      wl_list_insert.
      
      Use assert to control if the ivilayer->order.surface_list is empty.
      Signed-off-by: eucan's avatarEmre Ucan <eucan@de.adit-jv.com>
      [Pekka: wrapped the commit message]
      Reviewed-by: Pekka Paalanen's avatarPekka Paalanen <pekka.paalanen@collabora.co.uk>
      
      [Earlier version Reviewed-by: Nobuhiko Tanibata
      <NOBUHIKO_TANIBATA@xddp.denso.co.jp>]
      38fcf381
  11. 20 Aug, 2015 1 commit
  12. 07 Aug, 2015 1 commit
  13. 16 Jul, 2015 2 commits