Documentation

[imhotep82]

No description provided.

[tjaehnel]

These two additional functions are nice.
But the toHex() function that was there has been removed without any replacement.

[imhotep82]

These two additional functions are nice.
But the toHex() function that was there has been removed without any replacement.

numberToHex does the same thing. Could we write some tests to verify the functionality?

See here: https://leafo.net/guides/testing-lua-modules-with-github-actions.html

[tjaehnel]

numberToHex has only one parameter, I can't pass a length to it, so it can't do the same thing.
Also I like to have the name as short as possible because you need it all the time.

[imhotep82]

numberToHex has only one parameter, I can't pass a length to it, so it can't do the same thing.
Also I like to have the name as short as possible because you need it all the time.

Is this used for padding the values with zeros? The new function just converts the value to a HEX notation.

[lnzspraj]

You could turn the length argument into an optional argument:

function numberToHex(num, bytes)
	return string.format('%' .. (bytes and '0' .. bytes * 2 or '') .. 'X', num)
end