Hi hi,

> here are some unit tests for hash helpers (pdf-hash-add-bool,
> pdf-hash-add-i32 and pdf-hash-addu32).
> 

Some minor things:

+/* Copyright (C) 2008-2011 Free Software Foundation, Inc. */
New files should have listed only the current year in the copyright
line.

+#include <check.h>
+#include <pdf-test-common.h>
+/*
+ * Test: pdf_hash_add_bool_001
Please add an empty line between the last include and the beginning of
the test description.

And regarding the tests themselves:
 * For booleans, include tests adding TRUE and FALSE.
 * For u32s, include tests not only for the MAX u32, but also for the
minimum one (0), just for completeness.
 * For i32s, include tests not only for the MAX i32, but also for the
MIN i32.

Cheers!

-- 
Aleksander

Attachment: signature.asc
Description: This is a digitally signed message part

Reply via email to