[RFC] Define proper semantics for range() function - #10826

This is probably fine for possible values (long, array, string), but can we make this explicit, like ((1 << start_type) | (1 << end_type)) != MAY_BE_STRING?

Sure, but I don't think this would handle the case range('1', '9'); as both would be marked as IS_ARRAY

Not sure why I marked three lines instead of two, I was mainly referring to the if in 2842.

Either way, this still causes an issue if both inputs are string digits as start_type == IS_ARRAY and end_type == IS_ARRAY means that ((1 << start_type) | (1 << end_type)) will never be equal to MAY_BE_STRING.

I thought about using the MAY_BE_* constants and do some bit mask checks but found it simpler to just use a new value that is greater than IS_STRING. I can however tweak it if necessary.

I would also add test cases with string numbers bigger than '9', i.e. '12'. That should be also clear what to expect from such parameters.

Such a test exists in range_inputs_numeric_basic.phpt

I've split them up into more files

Sorry for the confusion. The description of the RFC is correct, however I meant test cases like range("12","A"). In your implementation it returns:

Warning: range(): Argument #1 ($start) must be a string if argument #2 ($end) is a string, argument #2 ($end) converted to 0 in Command line code on line 1
array(13) {
  [0]=>
  int(12)
  [1]=>
  int(11)
  [2]=>
  int(10)
  [3]=>
  int(9)
  [4]=>
  int(8)
  [5]=>
  int(7)
  [6]=>
  int(6)
  [7]=>
  int(5)
  [8]=>
  int(4)
  [9]=>
  int(3)
  [10]=>
  int(2)
  [11]=>
  int(1)
  [12]=>
  int(0)
}

The logic behind is arguable. I'm not sure exactly how it should behave for such case, but for sure the description is kind misleading. Because both of the arguments are string already.

range() can only work with single byte strings. Thus the behaviour of trying to interpret the numeric string as an int is the correct one. I will amend the error message to precise "single byte strings"