Comments (8)
What is the specific problem? Based on http://www.sqlite.org/c3ref/int64.html, it looks like the only way sqlite3_int64 could be less than 64-bits would be if someone would #define SQLITE_INT64_TYPE to be a type with less than 64-bits (ie. force the build to use something else than 64-bit default types). Why would anyone do that? This doesn't seem like something we should try to defend ourselves against.
from direct-sqlite.
If SQLITE_INT64_TYPE is not defined, sqlite3_int64 defaults to long
long it. Do you think direct-sqlite will ever run in a platform where
long long int is more than 64 bits (say, 128 bits)? If not, this is
not an issue.
2013/9/14, Janne Hellsten [email protected]:
What is the specific problem? Based on
http://www.sqlite.org/c3ref/int64.html, it looks like the only way
sqlite3_int64 could be less than 64-bits would be if someone would #define
SQLITE_INT64_TYPE to be a type with less than 64-bits (ie. force the build
to use something else than 64-bit default types). Why would anyone do that?
This doesn't seem like something we should try to defend ourselves
against.
Reply to this email directly or view it on GitHub:
#34 (comment)
from direct-sqlite.
I think you will have a hard time finding a platform where long long is larger than 64-bits. I think the SQLite defaults work just fine on platforms that support Haskell. I vote for closing this as not a bug.
from direct-sqlite.
Sure. I did find a message in a gcc c++ header saying "assume long
long no bigger than __int128". However, I could not find any platform,
supported by Haskell or not, where long long is supposed to be bigger
than 64 bits. There are even platforms where sqlite3 defines double as
integer. If you think it's paranoid to check for all these things, I
can't disagree.
2013/9/15, Janne Hellsten [email protected]:
I think you will have a hard time finding a platform where long long is
larger than 64-bits. I think the SQLite defaults work just fine on
platforms that support Haskell. I vote for closing this as not a bug.
Reply to this email directly or view it on GitHub:
#34 (comment)
from direct-sqlite.
I maintain that this should be closed as not a bug. If it happens that a platform with > 64-bits comes around (w/ long long != 64-bits), I'm sure the SQLite header with these typedefs will be fixed accordingly. Anyhow, if you're really interested in pursuing this (IMO theoretical) possibility further, perhaps take it up with the upstream SQLite project?
from direct-sqlite.
I agree, of course. Just close this as not a bug. If I ever find a
platform where this is a problem, the upstream sqlite code is what
should be updated. Thanks.
2013/9/16, Janne Hellsten [email protected]:
I maintain that this should be closed as not a bug. If it happens that a
platform with > 64-bits comes around (w/ long long != 64-bits), I'm sure the
SQLite header with these typedefs will be fixed accordingly. Anyhow, if
you're really interested in pursuing this (IMO theoretical) possibility
further, perhaps take it up with the upstream SQLite project?
Reply to this email directly or view it on GitHub:
#34 (comment)
from direct-sqlite.
I think only @IreneKnapp (the author) or you can close this. :)
from direct-sqlite.
Not a bug. Thank you all for the report and productive discussion, though!
from direct-sqlite.
Related Issues (20)
- Enable math functions HOT 3
- Add built-tool-depends hsc2hs? HOT 2
- Expose setting sqlite3_busy_timeout
- 🔥🧠
- How to add FTS5? HOT 5
- Any need to apply stat64 workaround? HOT 4
- Unreliable sqlErrorDetails in parallelized environments HOT 5
- Inaccurate version bound / build failure for GHC < 7.10 HOT 1
- Upgrading SQLite to v3.27.2
- Add a flag for compiling sqlite in Multi-thread mode HOT 3
- Upgrade embedded SQLite library HOT 2
- Test failure with system sqlite 3.34.1 HOT 1
- Nondeterministic `ErrorMisuse` on multiple runs of the same application test HOT 9
- Add Bindings to sqlite3_status and sqlite3_status64 HOT 1
- Slowness on `stepNoCB` HOT 1
- Updating test case for upgrading SQLite HOT 1
- Allow semigroup-0.20 HOT 1
- ICU extension flag HOT 3
- Allow opening sqllite from bytestring via sqllite3_deserialize directive HOT 4
- Can't use URI HOT 3
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from direct-sqlite.