Change "while (true)" inside Strings.sol

[BullshitLabProducts]

🧐 Motivation

@Amxx
Just an Idea, in the contract contracts/utils/Strings.sol there is a function called toString(uint256 value), which ends like:

while (true) {
    ptr--;
    assembly ("memory-safe") {
        mstore8(ptr, byte(mod(value, 10), HEX_DIGITS))
    }
    value /= 10;
    if (value == 0) break;
}
return buffer;

My idea is to change the logical condition to:

while (value != 0) {
    ptr--;
    assembly ("memory-safe") {
        mstore8(ptr, byte(mod(value, 10), HEX_DIGITS))
    }
    value /= 10;
}
return buffer;

Avoiding using break in loops helps minimize potential logical errors which may occur later on and it's best practice.

📝 Details

Change the logical condition of the while loop inside Strings.sol (please refer to the above)

[BullshitLabProducts]

Converted to a pull request
Thanks