Changes file extension of public facing files from .h to .hpp

AnswerDotAI / gpu.cpp

A lightweight library for portable low-level GPU computation using WebGPU.

https://gpucpp.answer.ai

Apache License 2.0

3.68k stars 175 forks source link

Changes file extension of public facing files from .h to .hpp #55

Closed OlekRaymond closed 2 weeks ago

OlekRaymond commented 3 weeks ago

The generally accepted assumptions are:

.h is the file extension for C header files
.hpp is the file extenstion for C++ header files

Currently all the C++ headers are marked as C header files. This makes it a little more confusing for tooling and people to understand how the code is intended to be used.

I have only made changes for files that are often user facing to minimise problems

More information is provided by Jason Turner

austinvhuang commented 3 weeks ago

This is reasonable to me, esp we may have some legit C header files for ABI interfaces in the future. It touches a lot of surface area, so I think we'll target it to the dev branch, merge and work out any kinks there before merging to main.

OlekRaymond commented 3 weeks ago

Shall do, thanks :)

OlekRaymond commented 2 weeks ago

@austinvhuang Did you want a new pull request or is the automatic rebase from github good?

austinvhuang commented 2 weeks ago

@OlekRaymond if you can resolve the merge conflicts we can go ahead and merge to the dev branch, either way works.