Re: [PATCH 0/5] Trivial randr cleanups

2015-03-20 Thread Alex Deucher
On Thu, Mar 19, 2015 at 9:01 AM, Emil Velikov emil.l.veli...@gmail.com wrote:
 Hello all,

 Noticed that as optimus support landed in randr, the latter became extra
 chatty, flooding the logs on said systems. Afaict the messages are not
 really meaningful outside the development stage, so I've decided to
 remove them. Alternatively we can change them to LogMessageVerb(X_INFO,
 3...) or similar.

 While there, my editor pointed out a few long lines so I've decided to
 address those as well :-)

 Any and all input will be appreciated.

For the series:

Reviewed-by: Alex Deucher alexander.deuc...@amd.com


 Cheers
 Emil


 Emil Velikov (5):
   randr: remove chatty error messages
   randr: use randr: prefix in ErrorF()
   randr: use local variables where possible
   randr: wrap long line
   randr: coding style fixes

  randr/rrcrtc.c   | 42 ++
  randr/rrscreen.c |  7 +++
  2 files changed, 25 insertions(+), 24 deletions(-)

 ___
 xorg-devel@lists.x.org: X.Org development
 Archives: http://lists.x.org/archives/xorg-devel
 Info: http://lists.x.org/mailman/listinfo/xorg-devel
___
xorg-devel@lists.x.org: X.Org development
Archives: http://lists.x.org/archives/xorg-devel
Info: http://lists.x.org/mailman/listinfo/xorg-devel

[PATCH 0/5] Trivial randr cleanups

2015-03-19 Thread Emil Velikov
Hello all,

Noticed that as optimus support landed in randr, the latter became extra 
chatty, flooding the logs on said systems. Afaict the messages are not 
really meaningful outside the development stage, so I've decided to 
remove them. Alternatively we can change them to LogMessageVerb(X_INFO, 
3...) or similar.

While there, my editor pointed out a few long lines so I've decided to 
address those as well :-)

Any and all input will be appreciated.

Cheers
Emil


Emil Velikov (5):
  randr: remove chatty error messages
  randr: use randr: prefix in ErrorF()
  randr: use local variables where possible
  randr: wrap long line
  randr: coding style fixes

 randr/rrcrtc.c   | 42 ++
 randr/rrscreen.c |  7 +++
 2 files changed, 25 insertions(+), 24 deletions(-)

___
xorg-devel@lists.x.org: X.Org development
Archives: http://lists.x.org/archives/xorg-devel
Info: http://lists.x.org/mailman/listinfo/xorg-devel