[ 
https://issues.apache.org/jira/browse/LUCENENET-231?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12776845#action_12776845
 ] 

George Aroush commented on LUCENENET-231:
-----------------------------------------

Ugh!  This was my bad.  I scripted out JLCA's "DateTime.Now.Ticks / 10000" 
variations to "System.DateTime.Now.Millisecond." without giving it full taught. 
 Sorry about that.

Instead of adding a new function to SupportClass, I rather see this fixed in 
the core doe directly.  SupportClass is a place to put stuff that if added in 
the core code will cause either extra noise in the code (large block of code), 
or code that isn't directly support in .NET.  So, if it's not too much work, I 
would rather see this fixed without the use of SupportClass.

> System.DateTime.Now.Millisecond bug
> -----------------------------------
>
>                 Key: LUCENENET-231
>                 URL: https://issues.apache.org/jira/browse/LUCENENET-231
>             Project: Lucene.Net
>          Issue Type: Bug
>         Environment: 2.9.0
>            Reporter: Digy
>         Attachments: CurrentTimeMillis.patch
>
>
> There are too many places where Java's System.currentTimeMillis() has been 
> ported as  System.DateTime.Now.Millisecond.
> I will replace them with a new method in SupportClass
> {code}
> public static long CurrentTimeMillis()
> {
>     return DateTime.Now.Ticks / 10000;
> }
> {code}

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to