Updating comments for deprecations.

The `@todo` tag is used for internal messages for developers maintaining the library and the class, explaining that (and why) it should be removed in a future version. The `@deprecated` tag is intended for the eyes of someone using this library to tell them that the method is deprecated and giving an alternative suggestion.

Author: brianmuse

src/Robin/Ntlm/Crypt/Random/McryptRandomByteGenerator.php

@@ -9,16 +9,16 @@
 namespace Robin\Ntlm\Crypt\Random;
 
 use Robin\Ntlm\Crypt\Exception\CryptographicFailureException;
-use UnexpectedValueException;
 
 /**
  * A cryptographically secure random byte generator implemented using the PHP
  * "mcrypt" extension.
  *
  * @link http://php.net/mcrypt
  * @deprecated This implementation is deprecated, as the mcrypt library
- *   is abandoned. More info: https://github.com/robinpowered/php-ntlm/pull/1
- * @todo Remove this implementation in a future version.
+ *   is abandoned. Use {@link NativeRandomByteGenerator} instead.
+ *   More info: https://github.com/robinpowered/php-ntlm/pull/1
+ * @todo Mcrypt is abandoned and this should be removed in a future version.
  */
 class McryptRandomByteGenerator implements RandomByteGeneratorInterface
 {
@@ -68,7 +68,7 @@ public function __construct($source = self::DEFAULT_SOURCE)
      * {@inheritDoc}
      *
      * @deprecated This implementation is deprecated, as the mcrypt
-     *   library is abandoned.
+     *   library is abandoned. Use {@link NativeRandomByteGenerator} instead.
      */
     public function generate($size)
     {

src/Robin/Ntlm/Crypt/Random/OpenSslRandomByteGenerator.php

@@ -15,9 +15,11 @@
  * "openssl" extension.
  *
  * @link http://php.net/openssl
- * @deprecated Insecure! This implementation is deprecated, as it's been found to
- *   be insecure. More info: https://github.com/robinpowered/php-ntlm/issues/7
- * @todo Remove this implementation in a future version.
+ * @deprecated This implementation is deprecated, as it's been found to
+ *   be insecure. Use {@link NativeRandomByteGenerator} instead.
+ *   More info: https://github.com/robinpowered/php-ntlm/issues/7
+ * @todo This random byte generator is insecure due to an issue with
+ *   `openssl_random_pseudo_bytes`. It should be removed in a future version.
  */
 class OpenSslRandomByteGenerator implements RandomByteGeneratorInterface
 {
@@ -29,8 +31,8 @@ class OpenSslRandomByteGenerator implements RandomByteGeneratorInterface
     /**
      * {@inheritDoc}
      *
-     * @deprecated Insecure! This implementation is deprecated, as it's been found
-     *   to be insecure.
+     * @deprecated This implementation is deprecated, as it's been found
+     *   to be insecure. Use {@link RandomByteGeneratorInterface} instead.
      */
     public function generate($size)
     {