Make the UPGRADING note about ArrayObject more explicit.

We noticed the BC break when testing `curl_setopt()`, and hadn't interpreted
this note as affecting internal functions in general, so this adds a sub-bullet
to note that.

This overlaps a bit with the previous bullet, but since I don't know exactly
which operations were in mind at the time, I've elected to preserve them both.
Bits are cheap.

diff --git a/UPGRADING b/UPGRADING
index 2a72976178a6241efd06280268e2ac6d997c9764..d9b46f2c381dab7b21016a5b96ea4b3fe4b7df85 100644 (file)
--- a/UPGRADING
+++ b/UPGRADING
@@ -120,6 +120,9 @@ PHP 7.4 UPGRADE NOTES
      * ReflectionObject::getProperties()
      * reset(), current(), etc. Use Iterator methods instead.
      * Potentially others working on object properties as a list.
+     * Other internal functions that iterate over an array, but which
+       previously silently accepted an ArrayObject as well; eg curl_setopt()
+       when used with an option that expects an array.
 
     (array) casts are *not* affected. They will continue to return either the
     wrapped array, or the ArrayObject properties, depending on whether the