Skip to content

Conversation

ghost
Copy link

@ghost ghost commented Aug 31, 2013

No description provided.

@Araq
Copy link
Member

Araq commented Sep 11, 2013

Float64 is now an alias for 'float' because you were right @ventor3000 and it was hard to work with. I am not sure what to do for ARM etc which prefers float32 for performance. I guess people can introduce their own "fastFloat" aliases.

@ventor3000
Copy link
Contributor

This is a good thing. In my taste, int should also always be an int32 (just the way modern c compilers works). This is because I hate code that works diffrently on diffrent platforms. In the unusual case where someone actullay want a type that has diffrent size on diffrent platforms, it's easy to define such a type with a simple when: statement. Also, this way the types cint and cdouble can be removed to slim stuff up.

@ghost
Copy link
Author

ghost commented Sep 13, 2013

Many people use int, which implies that they don't care what size it is. Is saving two characters here and there a big deal?

@ghost
Copy link
Author

ghost commented Sep 13, 2013

Float64 is now an alias for 'float' because you were right @ventor3000 and it was hard to work with. I am not sure what to do for ARM etc which prefers float32 for performance. I guess people can introduce their own "fastFloat" aliases.

So is one of those types redundant now? Might as well just make float a simple type alias in system.nim rather than having something like fastFloat. Or maybe it could be implemented temporarily if it's hard to change the way float works, and then a tool could deal with the migration.

@ghost ghost closed this Sep 13, 2013
@ghost ghost reopened this Sep 13, 2013
@ghost
Copy link
Author

ghost commented Sep 13, 2013

@ventor3000 I'm sure cint is 32 bits on many platforms, but surely it can vary.

@stevedonovan
Copy link

On Fri, Sep 13, 2013 at 1:12 PM, Erik Johansson Andersson <
notifications@github.com> wrote:
@ventor3000 https://github.com/ventor3000 I'm sure cint is 32 bits on
many platforms, but surely it can vary.

Well, sure. int is not a specified size in C, whereas float is.

@Araq
Copy link
Member

Araq commented Feb 2, 2014

please make this against devel and I'll merge

@Araq Araq closed this Feb 2, 2014
@ghost ghost deleted the op_type_params branch July 1, 2017 01:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants