An-DJ commented on PR #1885:
URL: https://github.com/apache/fury/pull/1885#issuecomment-2537665188

   Hi @chaokunyang 
   
   Recently I have searched and found that windows(MSVC) standards basic [data 
type 
range](https://learn.microsoft.com/en-us/cpp/cpp/data-type-ranges?view=msvc-170).
 Also in [intel 
docs](https://www.intel.com/content/www/us/en/developer/articles/technical/size-of-long-integer-type-on-different-architecture-and-os.html).
   
   Unfortunately, `long`, which used as `l` in Python, is stricted in 4 bytes 
on Windows, but 8 bytes on *nix. That is a gap between *nix and Windows.
   
   So Apache Fury support `q` (in another words, `signed long long`) ONLY maybe 
a better choice. That type is 8 bytes size in both *nix and Windows.
   
   I am not fimilair with Python array.array() users' habit, but this approach 
could be a simple solution to universe the memory model in Apache Fury Python 
support.
   
   What do you think of that?


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to