Skip to content

Handle 64-bit integers in Fortran properly#116

Merged
bast merged 1 commit into
dftlibs:masterfrom
robertodr:fortran-int-64
Feb 20, 2020
Merged

Handle 64-bit integers in Fortran properly#116
bast merged 1 commit into
dftlibs:masterfrom
robertodr:fortran-int-64

Conversation

@robertodr

Copy link
Copy Markdown
Contributor

This fixes compiler errors when setting default integer size to 64-bit in Fortran hosts.

Just a comment. I don't understand why the use of 64-bit integers in Fortran is a thing. But it poses an undue burden on library developers. And it is not fun. To any developer of a Fortran host to XCFun: this one commit is one hour of my life I will never get back.

@robertodr robertodr requested a review from bast February 20, 2020 11:00
@bast

bast commented Feb 20, 2020

Copy link
Copy Markdown
Member

Looking at it and I agree with the comment.

@foeroyingur

Copy link
Copy Markdown

Lots of problems in the host program as well... Btw there is nothing preventing library developers from sticking to 32-bit integers? Just have to make the proper conversion in the interface in the host program.

@bast

bast commented Feb 20, 2020

Copy link
Copy Markdown
Member

I agree but often it's the same poor souls who have to do the work.

@bast bast left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you so much!

@bast bast merged commit f0f8835 into dftlibs:master Feb 20, 2020
@robertodr robertodr deleted the fortran-int-64 branch February 20, 2020 12:27
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