You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@lucenenet.apache.org by ni...@apache.org on 2019/08/07 22:26:36 UTC

[lucenenet] 39/42: BUG: Lucene.Net.Util.StringHelper - Use Time.CurrentTimeMilliseconds() instead of DateTime.Now.Millisecond. The latter is a mis-translation from Java which contains only numbers 0 to 999, the former returns a long based on Stopwatch.GetTimestamp() that has several orders of magnitude more possible values.

This is an automated email from the ASF dual-hosted git repository.

nightowl888 pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/lucenenet.git

commit f0fb88fac6571f4c0a8fdf37803e6937d0f6c845
Author: Shad Storhaug <sh...@shadstorhaug.com>
AuthorDate: Wed Aug 7 11:12:09 2019 +0700

    BUG: Lucene.Net.Util.StringHelper - Use Time.CurrentTimeMilliseconds() instead of DateTime.Now.Millisecond. The latter is a mis-translation from Java which contains only numbers 0 to 999, the former returns a long based on Stopwatch.GetTimestamp() that has several orders of magnitude more possible values.
---
 src/Lucene.Net/Util/StringHelper.cs | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/Lucene.Net/Util/StringHelper.cs b/src/Lucene.Net/Util/StringHelper.cs
index 0dcbac2..459ee71 100644
--- a/src/Lucene.Net/Util/StringHelper.cs
+++ b/src/Lucene.Net/Util/StringHelper.cs
@@ -56,7 +56,7 @@ namespace Lucene.Net.Util
             }
             else
             {
-                return DateTime.Now.Millisecond;
+                return (int)Time.CurrentTimeMilliseconds();
             }
         }