Hello,
must comment two things!
--- Quote Start ---
In your code here you've specified also a clock in the sensitivity list but never used it in the code, remove it from the sensitivity list of the process.
--- Quote End ---
Apart from confusing the author himself and project coworkers, which effect would you expect particularly from superfluous sensitivity list members. I assume - none.
--- Quote Start ---
It'll be better if you not use the "*" but that you define the hardware multiplicator.
--- Quote End ---
Which purpose does it serve, except from making the code less readable?
I regard it a valuable feature of HDL compilers, that they are able to infer certain structures, including multipliers and memory from HDL code and generally try to utilize it for compact, functional code.
In the "no-output" example, you can doubt if multiplications and other long-winded operations are needed at all, but that's a different question.
Regards,
Frank