Update bson_ascii_strtoll() usage in Timestamp and UTCDateTime

bson_ascii_strtoll() resets errno, so that is no longer necessary. This also removes some outdated comments.

Author: jmikola

src/BSON/Timestamp.c

@@ -60,15 +60,8 @@ static bool php_phongo_timestamp_init_from_string(php_phongo_timestamp_t* intern
 	int64_t increment, timestamp;
 	char*   endptr = NULL;
 
-	errno = 0;
-
-	/* errno will set errno if conversion fails; however, we do not need to
-	 * specify the type of error.
-	 *
-	 * Note: bson_ascii_strtoll() does not properly detect out-of-range values
-	 * (see: CDRIVER-1377). strtoll() would be preferable, but it is not
-	 * available on all platforms (e.g. HP-UX), and atoll() provides no error
-	 * reporting at all. */
+	/* bson_ascii_strtoll() sets errno if conversion fails. If conversion
+	 * succeeds, we still want to ensure that the entire string was parsed. */
 
 	increment = bson_ascii_strtoll(s_increment, &endptr, 10);
 

src/BSON/UTCDateTime.c

@@ -54,17 +54,11 @@ static bool php_phongo_utcdatetime_init_from_string(php_phongo_utcdatetime_t* in
 	int64_t milliseconds;
 	char*   endptr = NULL;
 
-	errno = 0;
+	/* bson_ascii_strtoll() sets errno if conversion fails. If conversion
+	 * succeeds, we still want to ensure that the entire string was parsed. */
 
 	milliseconds = bson_ascii_strtoll(s_milliseconds, &endptr, 10);
 
-	/* errno will set errno if conversion fails; however, we do not need to
-	 * specify the type of error.
-	 *
-	 * Note: bson_ascii_strtoll() does not properly detect out-of-range values
-	 * (see: CDRIVER-1377). strtoll() would be preferable, but it is not
-	 * available on all platforms (e.g. HP-UX), and atoll() provides no error
-	 * reporting at all. */
 	if (errno || (endptr && endptr != ((const char*) s_milliseconds + s_milliseconds_len))) {
 		phongo_throw_exception(PHONGO_ERROR_INVALID_ARGUMENT TSRMLS_CC, "Error parsing \"%s\" as 64-bit integer for %s initialization", s_milliseconds, ZSTR_VAL(php_phongo_utcdatetime_ce->name));
 		return false;