diff mbox

[i-g-t] lib/core: Fix docs for igt_assert_lt(e)

Message ID 1443783419-21074-1-git-send-email-daniel.vetter@ffwll.ch (mailing list archive)
State New, archived
Headers show

Commit Message

Daniel Vetter Oct. 2, 2015, 10:56 a.m. UTC 
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(-)

Comments

Daniel Stone Oct. 7, 2015, 3:52 p.m. UTC | #1 
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
Daniel Vetter Oct. 7, 2015, 4:25 p.m. UTC | #2 
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
diff mbox

Patch

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