On Thu, Nov 17, 2016 at 04:42:07PM -0800, Bryce Harrington wrote: > On Thu, Nov 10, 2016 at 03:02:06PM +1000, Peter Hutterer wrote: > > 8 spaces changed to one tab > > > > Signed-off-by: Peter Hutterer <peter.hutte...@who-t.net> > > The patch isn't applying as of change 66a26aeb (remove inconsistent line > breaks), but in generating a whitespace patch myself using emacs with > the Wayland style rules, I am getting a similar looking set of changes. > While whitespace changes are trivial, having these fixed would make > automatic style tools a bit better able to detect and fix whitespace > issues in the future.
do we have any plans for using those? The simplest case, detecting 8 spaces where there should be a tab is a 3-line hack in the Makefile.am Cheers, Peter > > Acked-by: Bryce Harrington <br...@osg.samsung.com> > > > --- > > protocol/wayland.xml | 100 > > +++++++++++++++++++++++++-------------------------- > > 1 file changed, 50 insertions(+), 50 deletions(-) > > > > diff --git a/protocol/wayland.xml b/protocol/wayland.xml > > index 6c6d078..bcc228f 100644 > > --- a/protocol/wayland.xml > > +++ b/protocol/wayland.xml > > @@ -129,7 +129,7 @@ > > <request name="bind"> > > <description summary="bind an object to the display"> > > Binds a new, client-created object to the server using the > > - specified name as the identifier. > > + specified name as the identifier. > > </description> > > <arg name="name" type="uint" summary="unique numeric name of the > > object"/> > > <arg name="id" type="new_id" summary="bounded object"/> > > @@ -139,9 +139,9 @@ > > <description summary="announce global object"> > > Notify the client of global objects. > > > > - The event notifies the client that a global object with > > - the given name is now available, and it implements the > > - given version of the given interface. > > + The event notifies the client that a global object with > > + the given name is now available, and it implements the > > + given version of the given interface. > > </description> > > <arg name="name" type="uint" summary="numeric name of the global > > object"/> > > <arg name="interface" type="string" summary="interface implemented > > by the object"/> > > @@ -152,10 +152,10 @@ > > <description summary="announce removal of global object"> > > Notify the client of removed global objects. > > > > - This event notifies the client that the global identified > > - by name is no longer available. If the client bound to > > - the global using the bind request, the client should now > > - destroy that object. > > + This event notifies the client that the global identified > > + by name is no longer available. If the client bound to > > + the global using the bind request, the client should now > > + destroy that object. > > > > The object remains valid and requests to the object will be > > ignored until the client destroys it, to avoid races between > > @@ -357,7 +357,7 @@ > > > > The pool can be used to create shared memory based buffer > > objects. The server will mmap size bytes of the passed file > > - descriptor, to use as backing memory for the pool. > > + descriptor, to use as backing memory for the pool. > > </description> > > > > <arg name="id" type="new_id" interface="wl_shm_pool" summary="pool > > to create"/> > > @@ -924,14 +924,14 @@ > > > > <request name="create_data_source"> > > <description summary="create a new data source"> > > - Create a new data source. > > + Create a new data source. > > </description> > > <arg name="id" type="new_id" interface="wl_data_source" > > summary="data source to create"/> > > </request> > > > > <request name="get_data_device"> > > <description summary="create a new data device"> > > - Create a new data device for a given seat. > > + Create a new data device for a given seat. > > </description> > > <arg name="id" type="new_id" interface="wl_data_device" > > summary="data device to create"/> > > <arg name="seat" type="object" interface="wl_seat" summary="seat > > associated with the data device"/> > > @@ -1320,7 +1320,7 @@ > > > > <enum name="error"> > > <description summary="wl_surface error values"> > > - These errors can be emitted in response to wl_surface requests. > > + These errors can be emitted in response to wl_surface requests. > > </description> > > <entry name="invalid_scale" value="0" summary="buffer scale value is > > invalid"/> > > <entry name="invalid_transform" value="1" summary="buffer transform > > value is invalid"/> > > @@ -1679,8 +1679,8 @@ > > > > <enum name="capability" bitfield="true"> > > <description summary="seat capability bitmask"> > > - This is a bitmask of capabilities this seat has; if a member is > > - set, then it is present on the seat. > > + This is a bitmask of capabilities this seat has; if a member is > > + set, then it is present on the seat. > > </description> > > <entry name="pointer" value="1" summary="the seat has pointer > > devices"/> > > <entry name="keyboard" value="2" summary="the seat has one or more > > keyboards"/> > > @@ -1878,7 +1878,7 @@ > > > > <enum name="button_state"> > > <description summary="physical button state"> > > - Describes the physical state of a button that produced the button > > + Describes the physical state of a button that produced the button > > event. > > </description> > > <entry name="released" value="0" summary="the button is not > > pressed"/> > > @@ -1891,8 +1891,8 @@ > > > > The location of the click is given by the last motion or > > enter event. > > - The time argument is a timestamp with millisecond > > - granularity, with an undefined base. > > + The time argument is a timestamp with millisecond > > + granularity, with an undefined base. > > </description> > > > > <arg name="serial" type="uint" summary="serial number of the button > > event"/> > > @@ -2106,7 +2106,7 @@ > > <entry name="no_keymap" value="0" > > summary="no keymap; client must understand how to interpret the > > raw keycode"/> > > <entry name="xkb_v1" value="1" > > - summary="libxkbcommon compatible; to determine the xkb > > keycode, clients must add 8 to the key event keycode"/> > > + summary="libxkbcommon compatible; to determine the xkb keycode, > > clients must add 8 to the key event keycode"/> > > </enum> > > > > <event name="keymap"> > > @@ -2152,8 +2152,8 @@ > > <event name="key"> > > <description summary="key event"> > > A key was pressed or released. > > - The time argument is a timestamp with millisecond > > - granularity, with an undefined base. > > + The time argument is a timestamp with millisecond > > + granularity, with an undefined base. > > </description> > > > > <arg name="serial" type="uint" summary="serial number of the key > > event"/> > > @@ -2185,24 +2185,24 @@ > > > > <event name="repeat_info" since="4"> > > <description summary="repeat rate and delay"> > > - Informs the client about the keyboard's repeat rate and delay. > > + Informs the client about the keyboard's repeat rate and delay. > > > > - This event is sent as soon as the wl_keyboard object has been > > created, > > - and is guaranteed to be received by the client before any key press > > - event. > > + This event is sent as soon as the wl_keyboard object has been created, > > + and is guaranteed to be received by the client before any key press > > + event. > > > > - Negative values for either rate or delay are illegal. A rate of > > zero > > - will disable any repeating (regardless of the value of delay). > > + Negative values for either rate or delay are illegal. A rate of zero > > + will disable any repeating (regardless of the value of delay). > > > > - This event can be sent later on as well with a new value if > > necessary, > > - so clients should continue listening for the event past the > > creation > > - of wl_keyboard. > > + This event can be sent later on as well with a new value if necessary, > > + so clients should continue listening for the event past the creation > > + of wl_keyboard. > > </description> > > > > <arg name="rate" type="int" > > - summary="the rate of repeating keys in characters per second"/> > > + summary="the rate of repeating keys in characters per second"/> > > <arg name="delay" type="int" > > - summary="delay in milliseconds since key down until repeating > > starts"/> > > + summary="delay in milliseconds since key down until repeating > > starts"/> > > </event> > > </interface> > > > > @@ -2372,10 +2372,10 @@ > > mode that was received with the current flag set. > > > > The size of a mode is given in physical hardware units of > > - the output device. This is not necessarily the same as > > - the output size in the global compositor space. For instance, > > - the output may be scaled, as described in wl_output.scale, > > - or transformed, as described in wl_output.transform. > > + the output device. This is not necessarily the same as > > + the output size in the global compositor space. For instance, > > + the output may be scaled, as described in wl_output.scale, > > + or transformed, as described in wl_output.transform. > > </description> > > <arg name="flags" type="uint" enum="mode" summary="bitfield of mode > > flags"/> > > <arg name="width" type="int" summary="width of the mode in hardware > > units"/> > > @@ -2387,20 +2387,20 @@ > > > > <event name="done" since="2"> > > <description summary="sent all information about output"> > > - This event is sent after all other properties have been > > - sent after binding to the output object and after any > > - other property changes done after that. This allows > > - changes to the output properties to be seen as > > - atomic, even if they happen via multiple events. > > + This event is sent after all other properties have been > > + sent after binding to the output object and after any > > + other property changes done after that. This allows > > + changes to the output properties to be seen as > > + atomic, even if they happen via multiple events. > > </description> > > </event> > > > > <event name="scale" since="2"> > > <description summary="output scaling properties"> > > This event contains scaling geometry information > > - that is not in the geometry event. It may be sent after > > - binding the output object or if the output scale changes > > - later. If it is not sent, the client should assume a > > + that is not in the geometry event. It may be sent after > > + binding the output object or if the output scale changes > > + later. If it is not sent, the client should assume a > > scale of 1. > > > > A scale larger than 1 means that the compositor will > > @@ -2499,7 +2499,7 @@ > > > > <enum name="error"> > > <entry name="bad_surface" value="0" > > - summary="the to-be sub-surface is invalid"/> > > + summary="the to-be sub-surface is invalid"/> > > </enum> > > > > <request name="get_subsurface"> > > @@ -2514,11 +2514,11 @@ > > </description> > > > > <arg name="id" type="new_id" interface="wl_subsurface" > > - summary="the new sub-surface object ID"/> > > + summary="the new sub-surface object ID"/> > > <arg name="surface" type="object" interface="wl_surface" > > - summary="the surface to be turned into a sub-surface"/> > > + summary="the surface to be turned into a sub-surface"/> > > <arg name="parent" type="object" interface="wl_surface" > > - summary="the parent surface"/> > > + summary="the parent surface"/> > > </request> > > </interface> > > > > @@ -2587,7 +2587,7 @@ > > > > <enum name="error"> > > <entry name="bad_surface" value="0" > > - summary="wl_surface is not a sibling or the parent"/> > > + summary="wl_surface is not a sibling or the parent"/> > > </enum> > > > > <request name="set_position"> > > @@ -2634,7 +2634,7 @@ > > </description> > > > > <arg name="sibling" type="object" interface="wl_surface" > > - summary="the reference surface"/> > > + summary="the reference surface"/> > > </request> > > > > <request name="place_below"> > > @@ -2644,7 +2644,7 @@ > > </description> > > > > <arg name="sibling" type="object" interface="wl_surface" > > - summary="the reference surface"/> > > + summary="the reference surface"/> > > </request> > > > > <request name="set_sync"> > > -- > > 2.9.3 > > > > _______________________________________________ > > wayland-devel mailing list > > wayland-devel@lists.freedesktop.org > > https://lists.freedesktop.org/mailman/listinfo/wayland-devel _______________________________________________ wayland-devel mailing list wayland-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/wayland-devel