awslabs / aws-crt-php

Apache License 2.0
322 stars 13 forks source link

Fix deprecated use of ${var} in Encoding.php #108

Closed mdio closed 8 months ago

mdio commented 8 months ago

Since PHP 8.2 the syntax "${var}" is deprecated. "{$var}" should be used instead.

Issue #, if available: #105

Description of changes: Changing "a${len}" to "a{$len}" works the same as before but doesn't use the ${var} syntax deprecated in PHP 8.2

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

waahm7 commented 8 months ago

LGTM! Thank you for the contribution.