64 bit string offsets

[email protected] ("Anatol Belski")
Newsgroups php.internals
Message-ID <[email protected]>
Hi,

while refining the big string support, it turned out that we've an issue.
The syntax like $s[42] = 'x'; is currently inconsistend, because we have
uint32 for string offsets. This actually means, the behaviour is currently
only available in the old style and can handle not more than 2gb big
strings.

Also discussed with Laruence on IRC and he actually expressed the concern
that we pay overhead on that. From my side I was investigating on that and
could suggest several solutions for that:

- stay with the old behavior (indexes would be able to handle only 2gb
strings, this is the status quo)
- implement a function like in JS String.charAt() as alternative
- turn to some temp_variable solution we currently have in PHP5. Laruence
told be that dropping temp_variable was one of the improvementes.
Actually, the string index functionality is utilized in two opcodes, so
maybe that were just a local case.

Anyway not talking about touching zval, as that would grow by 8 bytes with
a size_t str_offset. Just maybe there were another solution I oversee?

Regards

Anatol
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.