search

lclevy / ADFlib

A free, portable and open implementation of the Amiga filesystem
GNU General Public License v2.0
84 stars 29 forks source link

Fixed typos in the headers and one incorrect function signature #78

Closed DidierMalenfant closed 6 months ago

DidierMalenfant commented 6 months ago

I also just added a fix for a warning in Release builds

t-w commented 6 months ago

Hi. Thanks for correcting these.

However, I would like to ask you that we close this PR. I have a lot of new/changed stuff in my fork (t-w/ADFlib) where further development is and probably will be done so that Laurent (@lclevy) is not bothered that much with PRs and stuff.

(I know there is no info about it yet... I will ask him to put info about it in the project description).

The latest (working) code should be, like here, in the devel branch. So if you open a PR to that one, I can merge it (I have just merge the latest stuff there).

(Merging this here would require a lot work with resolving conflicts, some modified files may have been renamed etc. so, it will be easier to update it on your side than mine...).

DidierMalenfant commented 6 months ago

So is your fork the main repo for the project now?

t-w commented 6 months ago

At the moment, in practice, that is the main development repo, while here goes things for release. We will see if we move everything (releases) there or keep them here.

Anyway, to go on with this one - I pushed the current stuff to the refactor branch in this repo. You can pull it, make your changes and update your devel so that the changes appear here.

DidierMalenfant commented 6 months ago

You let me know what you prefer, I don't want it to get too confusing...

I've started making these changes for your other repo as requested, I'll close this one and submit another one there.