On 5/10/13 02:45 , Ping Cheng wrote:
> On Thu, Oct 3, 2013 at 9:59 PM, Peter Hutterer <peter.hutte...@who-t.net
> <mailto:peter.hutte...@who-t.net>> wrote:
>
>     On Thu, Oct 03, 2013 at 05:17:48PM -0700, Ping Cheng wrote:
>      > And wcmThreshold applies to pen (stylus and eraser) tools.
>      >
>      > Signed-off-by: Ping Cheng <pi...@wacom.com <mailto:pi...@wacom.com>>
>      > ---
>
>     please swap the two commit sentences around. the bigger change here
>     is that
>     the message is for all pen devices now, the removal of the redundant
>     name is
>     just a side-issue
>
>     remember that the first line of the commit message is the one people see
>     first (or even the only line they see when searching), so it should
>     always
>     contain the more important change.
>
>     Reviewed-by: Peter Hutterer <peter.hutte...@who-t.net
>     <mailto:peter.hutte...@who-t.net>>
>     for the code though.
>
>
> Do you mean just for this patch or the other two (2/4 and 3/4) as well?
> I need to know if I should wait for more comments about this series or not.

I'll try to review the others on monday, ran out of time on Fri before I 
had to head off, sorry.

Cheers,
   Peter

>
>
>      >  src/wcmCommon.c | 12 +++++-------
>      >  1 file changed, 5 insertions(+), 7 deletions(-)
>      >
>      > diff --git a/src/wcmCommon.c b/src/wcmCommon.c
>      > index c81310f..df71842 100644
>      > --- a/src/wcmCommon.c
>      > +++ b/src/wcmCommon.c
>      > @@ -1323,7 +1323,7 @@ int wcmInitTablet(InputInfoPtr pInfo, const
>     char* id, float version)
>      >               return !Success;
>      >
>      >       /* Default threshold value if not set */
>      > -     if (common->wcmThreshold <= 0)
>      > +     if (common->wcmThreshold <= 0 && IsPen(priv))
>      >       {
>      >               /* Threshold for counting pressure as a button */
>      >               common->wcmThreshold = DEFAULT_THRESHOLD;
>      > @@ -1333,19 +1333,17 @@ int wcmInitTablet(InputInfoPtr pInfo,
>     const char* id, float version)
>      >       }
>      >
>      >       /* output tablet state as probed */
>      > -     if (TabletHasFeature(common, WCM_PEN))
>      > -             xf86Msg(X_PROBED, "%s: Wacom %s tablet maxX=%d
>     maxY=%d maxZ=%d "
>      > +     if (IsPen(priv))
>      > +             xf86Msg(X_PROBED, "%s: maxX=%d maxY=%d maxZ=%d "
>      >                       "resX=%d resY=%d  tilt=%s\n",
>      >                       pInfo->name,
>      > -                     model->name,
>      >                       common->wcmMaxX, common->wcmMaxY,
>     common->wcmMaxZ,
>      >                       common->wcmResolX, common->wcmResolY,
>      >                       HANDLE_TILT(common) ? "enabled" : "disabled");
>      > -     else
>      > -             xf86Msg(X_PROBED, "%s: Wacom %s tablet maxX=%d
>     maxY=%d maxZ=%d "
>      > +     else if (IsTouch(priv))
>      > +             xf86Msg(X_PROBED, "%s: maxX=%d maxY=%d maxZ=%d "
>      >                       "resX=%d resY=%d \n",
>      >                       pInfo->name,
>      > -                     model->name,
>      >                       common->wcmMaxTouchX, common->wcmMaxTouchY,
>      >                       common->wcmMaxZ,
>      >                       common->wcmTouchResolX,
>     common->wcmTouchResolY);
>      > --
>      > 1.8.1.2
>      >
>
>


------------------------------------------------------------------------------
October Webinars: Code for Performance
Free Intel webinars can help you accelerate application performance.
Explore tips for MPI, OpenMP, advanced profiling, and more. Get the most from 
the latest Intel processors and coprocessors. See abstracts and register >
http://pubads.g.doubleclick.net/gampad/clk?id=60134791&iu=/4140/ostg.clktrk
_______________________________________________
Linuxwacom-devel mailing list
Linuxwacom-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/linuxwacom-devel

Reply via email to