braedonwooding/cpath

A simple and nice looking cross platform C FileSystem with full C++ support

Hi there 👋 🤗

• 💼 I'm currently working at Resonate Solutions
• 🔭 I’m currently interested in Compilers and Databases
• 😄 Pronouns: he/him or they/them

---

Recent Projects

• Tau Time A time math library for Typescript
• Quaint A painting application in Java
• ChonkyStateMachine A simple state machine in C#
• ChonkyReviews A google reviews sandbox/testing mock application

cpath's Issues

Investigate dynamically sized paths

This will make some APIs more optimal at the cost of more allocations.

For example, it'll reduce the number of copies we have to do in cpathPeekNextFile.

This would also let us skip doing str length checks.

Rewrite API

There are a few points in the API that are pretty messy.

• Definition of constants
• Lack of GDB/LLDB support for our CPP strings
• Support C++ in windows
• Max Path Length in windows isn't always 256, you can for example use a \\?\ prefix in older windows to allow longer paths (up to 32kbs ish).
  • I'm not going to support UNC paths (the \\?\ prefix) though (let's document that though).
  • I don't want to support an absurd stack array of anything above a couple thousand bytes. So we should require a buffer to be passed in for the cases of long paths, but we should allow overriding the path length for the case of windows.

In cpathGetNextFile, we check if file == NULL, however, file is not assigned as NULL upon initialisation

If you run anything that calls cpathGetNextFile in valgrind, it will also let you know about this issue. The problem is here:

[Windows] cpathCanonicalise: doesn't set length correctly

cpath/cpath.h

Line 1064 in ff6bfae

path->len = size;

Sets the input path.

Also I'm not entirely sure why we are handling the case of non-restrict, I doubt that'll apply to both windows & linux so we can likely skip a copy on one of those cases.

Likely other places too.

Add if: always() to github tests

To force the tests to pass

CPath Fix Tests

For some reason I've got some focus to push some changes on this...

So going to work on fixing the failing tests!