Intel-GFX Archive on lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH i-g-t] lib/core: Fix docs for igt_assert_lt(e)
@ 2015-10-02 10:56 Daniel Vetter
  2015-10-07 15:52 ` Daniel Stone
  0 siblings, 1 reply; 3+ messages in thread
From: Daniel Vetter @ 2015-10-02 10:56 UTC (permalink / raw)
  To: Intel Graphics Development; +Cc: Daniel Vetter, Daniel Vetter

Logical negation is hard.

Cc: "Morton, Derek J" <derek.j.morton@intel.com>
Signed-off-by: Daniel Vetter <daniel.vetter@intel.com>
---
 lib/igt_core.h | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/lib/igt_core.h b/lib/igt_core.h
index f8bfbf0145e1..293f691d0025 100644
--- a/lib/igt_core.h
+++ b/lib/igt_core.h
@@ -458,7 +458,7 @@ void igt_exit(void) __attribute__((noreturn));
  * @n1: first integer
  * @n2: second integer
  *
- * Fails (sub-)test if the second integers is greater than the first.
+ * Fails (sub-)test if the second integers is strictly smaller than the first.
  * Beware that for now this only works on integers.
  *
  * Like igt_assert(), but displays the values being compared on failure instead
@@ -471,7 +471,7 @@ void igt_exit(void) __attribute__((noreturn));
  * @n1: first integer
  * @n2: second integer
  *
- * Fails (sub-)test if the second integers is strictly smaller than the first.
+ * Fails (sub-)test if the second integers is smaller than or equal to the first.
  * Beware that for now this only works on integers.
  *
  * Like igt_assert(), but displays the values being compared on failure instead
-- 
2.5.1

_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx

^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [PATCH i-g-t] lib/core: Fix docs for igt_assert_lt(e)
  2015-10-02 10:56 [PATCH i-g-t] lib/core: Fix docs for igt_assert_lt(e) Daniel Vetter
@ 2015-10-07 15:52 ` Daniel Stone
  2015-10-07 16:25   ` Daniel Vetter
  0 siblings, 1 reply; 3+ messages in thread
From: Daniel Stone @ 2015-10-07 15:52 UTC (permalink / raw)
  To: Daniel Vetter; +Cc: Daniel Vetter, Intel Graphics Development

Hi,

On 2 October 2015 at 11:56, Daniel Vetter <daniel.vetter@ffwll.ch> wrote:
> Logical negation is hard.
>
> Cc: "Morton, Derek J" <derek.j.morton@intel.com>
> Signed-off-by: Daniel Vetter <daniel.vetter@intel.com>

Almost identical to what I had locally.

s/second integers/second integer/ and:
Reviewed-by: Daniel Stone <daniels@collabora.com>

Cheers,
Daniel
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH i-g-t] lib/core: Fix docs for igt_assert_lt(e)
  2015-10-07 15:52 ` Daniel Stone
@ 2015-10-07 16:25   ` Daniel Vetter
  0 siblings, 0 replies; 3+ messages in thread
From: Daniel Vetter @ 2015-10-07 16:25 UTC (permalink / raw)
  To: Daniel Stone; +Cc: Daniel Vetter, Intel Graphics Development, Daniel Vetter

On Wed, Oct 07, 2015 at 04:52:46PM +0100, Daniel Stone wrote:
> Hi,
> 
> On 2 October 2015 at 11:56, Daniel Vetter <daniel.vetter@ffwll.ch> wrote:
> > Logical negation is hard.
> >
> > Cc: "Morton, Derek J" <derek.j.morton@intel.com>
> > Signed-off-by: Daniel Vetter <daniel.vetter@intel.com>
> 
> Almost identical to what I had locally.
> 
> s/second integers/second integer/ and:
> Reviewed-by: Daniel Stone <daniels@collabora.com>

Fixed and applied, thanks.
-Daniel

> 
> Cheers,
> Daniel

-- 
Daniel Vetter
Software Engineer, Intel Corporation
http://blog.ffwll.ch
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2015-10-07 16:22 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-10-02 10:56 [PATCH i-g-t] lib/core: Fix docs for igt_assert_lt(e) Daniel Vetter
2015-10-07 15:52 ` Daniel Stone
2015-10-07 16:25   ` Daniel Vetter

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox