Skip to content

Fix minor typos #60

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 3 commits into from
Jan 17, 2024
Merged

Fix minor typos #60

merged 3 commits into from
Jan 17, 2024

Conversation

m8t
Copy link

@m8t m8t commented Jan 17, 2024

Found minor things to adjust within tools/cli/modules.md.

  • Replace old URL for list of proposed software (currently points to old documentation)
  • s/ifortran/ifort/ the compiler from Intel for Fortan language
  • Fortran file extension .f90 does not require special space prefixes
  • No need to compile and link mpi_broadcast.c with pthread, it's done automatically (with many other libs)
  • Remove useless include in MPI example

Mike MASSONNET added 3 commits January 17, 2024 13:31
Fix typos
737ea39 
- Minor typos
- Replace old URL for list of proposed software
- s/ifortran/ifort/ the compiler from Intel for Fortan language
- No need to compile and link mpi_broadcast.c with pthread, it's done automatically (with many other libs)
With Fortran .f90 extension no need for prefixed spaces
1cf7846 
.f90 is the standard file extension for modern Fortran source files.
The 90 refers to the first modern Fortran standard in 1990.
MPI sample program compiles without header time.h
aadf521
@hcartiaux hcartiaux merged commit 6fbd8bf into devel Jan 17, 2024
@m8t m8t deleted the m8t/fix-typos branch January 17, 2024 16:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@m8t @hcartiaux