¿Qué tiene de malo este código que usa la extensión mysql para obtener datos de una base de datos en PHP?

Las variables en php distinguen entre mayúsculas y minúsculas. Reemplace su ciclo while con lo siguiente:

while ($rows = mysql_fetch_array($query)):

           $name = $rows['Name'];
           $address = $rows['Address'];
           $email = $rows['Email'];
           $subject = $rows['Subject'];
           $comment = $rows['Comment']

           echo "$name<br>$address<br>$email<br>$subject<br>$comment<br><br>";

           endwhile;