-
Notifications
You must be signed in to change notification settings - Fork 41
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
ADIOS_DATATYPES missing long long #153
Comments
adios_long is always 64bit
adios_integer is always 32bit
no exceptions
…On Tue, Nov 7, 2017 at 10:36 AM Burlen Loring ***@***.***> wrote:
not sure if this is intentional or not, but from the perspective of trying
to integrate other library's data structures which make use of long long
with ADIOS it seems to be an omission. according to c99 long is at least 32
bits and long long is at least 64 bits, so there could be (admittedly rare)
cases when the two are not the same size.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
<#153>, or mute the thread
<https://github.com/notifications/unsubscribe-auth/ADGMLZdMxRtcc9MAIZUO3MVLehnnYVqVks5s0JUxgaJpZM4QVLHq>
.
|
OK, works for me |
Although I wonder then what happens on a system where long is 32 bits? My inclination would be to always map long to adios_long. But in ADIOS API void* is used, and it would catastrophic to pass a long(32 bits) and tell ADIOS it's adios_long(64 bits). I think ADIOS is doing a confusing thing here. |
For instance here's a code snippet
I will have to modify to this
|
so rather than being equivalent their C counterparts, |
not sure if this is intentional or not, but from the perspective of trying to integrate other library's data structures which make use of long long with ADIOS it seems to be an omission. according to c99 long is at least 32 bits and long long is at least 64 bits, so there could be (admittedly rare) cases when the two are not the same size.
The text was updated successfully, but these errors were encountered: