Looks ok, but now the above few lines of code do both sizeof(int) and sizeof(<variable name>).
Should be consistent, I think better as it was (sizeof(e))