Fixes np_short error message

Author: sobolevn

Lib/test/test_struct.py

@@ -679,14 +679,23 @@ def test_issue35714(self):
  struct.calcsize(s)
 
  @support.cpython_only
- def test_issue45034(self):
+ def test_issue45034_unsigned(self):
  from _testcapi import USHRT_MAX
  error_msg = f'ushort format requires 0 <= number <= {USHRT_MAX}'
  with self.assertRaisesRegex(struct.error, error_msg):
  struct.pack('H', 70000) # too large
  with self.assertRaisesRegex(struct.error, error_msg):
  struct.pack('H', -1) # too small
 
+ @support.cpython_only
+ def test_issue45034_signed(self):
+ from _testcapi import SHRT_MIN, SHRT_MAX
+ error_msg = f'short format requires {SHRT_MIN} <= number <= {SHRT_MAX}'
+ with self.assertRaisesRegex(struct.error, error_msg):
+ struct.pack('h', 70000) # too large
+ with self.assertRaisesRegex(struct.error, error_msg):
+ struct.pack('h', -70000) # too small
+
 
 class UnpackIteratorTest(unittest.TestCase):
  """

Misc/NEWS.d/next/Library/2021-09-05-20-33-25.bpo-45034.62NLD5.rst

@@ -1,2 +1,2 @@
-Changes how max value is formatted for ``struct.pack`` with ``'H'`` mode and
-too large / small numbers.
+Changes how error is formatted for ``struct.pack`` with ``'H'`` and ``'h'`` modes and
+too large / small numbers. Now it shows the actual numeric limits, while previously it was showing arithmetic expressions.

Modules/_struct.c

@@ -589,9 +589,10 @@ np_short(_structmodulestate *state, char *p, PyObject *v, const formatdef *f)
  if (get_long(state, v, &x) < 0)
  return -1;
  if (x < SHRT_MIN || x > SHRT_MAX) {
- PyErr_SetString(state->StructError,
- "short format requires " Py_STRINGIFY(SHRT_MIN)
- " <= number <= " Py_STRINGIFY(SHRT_MAX));
+ PyErr_Format(state->StructError,
+ "short format requires %d <= number <= %d",
+ SHRT_MIN,
+ SHRT_MAX);
  return -1;
  }
  y = (short)x;
@@ -609,7 +610,7 @@ np_ushort(_structmodulestate *state, char *p, PyObject *v, const formatdef *f)
  if (x < 0 || x > USHRT_MAX) {
  PyErr_Format(state->StructError,
  "ushort format requires 0 <= number <= %u",
- (unsigned int)(USHRT_MAX));
+ (unsigned int)USHRT_MAX);
  return -1;
  }
  y = (unsigned short)x;