review tweaks

Author: kilograham

src/rp2_common/pico_double/include/pico/double.h

@@ -95,14 +95,10 @@ extern "C" {
 *
 *     int2double, uint2double, int642double, uint642double
 *
-*     note: on `pico_double_vfp` the 32-bit functions are also provided as C macros too since they map to inline VFP code
-*
 *   - (u)double -> int (round towards zero):
 *
 *     double2int_z, double2uint_z, double2int64_z, double2uint64_z
 *
-*     note: on `pico_double_vfp` the 32-bit functions are also provided as C macros too since they map to inline VFP code
-*
 *   - (u)double -> int (round towards -infinity):
 *
 *     double2int, double2uint, double2int64, double2uint64
@@ -113,23 +109,14 @@ extern "C" {
 *
 *       fix2double, ufix2double, fix642double, ufix642double
 *
-*     note: on `pico_double_vfp` the 32-bit functions are currently _only_ provided as C macros and must use a compile
-*     time constant between 1 and 32 for the fixed point position
-*
 *   - double -> (u)fix (round towards zero):
 *
 *       double2fix_z, double2ufix_z, double2fix64_z, double2ufix64_z
 *
-*     note: on `pico_double_vfp` the 32-bit functions are currently _only_ provided as C macros and must use a compile
-*     time constant between 1 and 32 for the fixed point position
-*
 *   - double -> (u)fix (round towards -infinity):
 *
 *       double2fix, double2ufix, double2fix64, double2ufix64
 *
-*     note: on `pico_double_vfp` the 32-bit functions are currently _only_ provided as C macros and must use a compile
-*     time constant between 1 and 32 for the fixed point position
-*
 * - Even faster versions of divide and square-root functions that do not round correctly:
 *
 *   ddiv_fast, sqrt_fast (these do not round correctly)

test/pico_float_test/custom_double_funcs_test.c

@@ -242,7 +242,7 @@ int test() {
     test_checki(double2int(-1.0), -1, "double2int5b");
     // todo test correct rounding around maximum precision
     test_checki(double2int(2147483646.0), INT32_MAX-1, "double2int6");
-    test_checki(double2int(2147483647.0), INT32_MAX, "double2int6");
+    test_checki(double2int(2147483647.0), INT32_MAX, "double2int6b");
     test_checki(double2int(21474836470.0), INT32_MAX, "double2int7");
     test_checki(double2int(-2147483648.0), INT32_MIN, "double2int8");
     test_checki(double2int(-21474836480.0), INT32_MIN, "double2int9");