djkoloski / fe Goto Github PK

Broken defaults:

We currently need to manually configure the Makey's project properties to enable us to debug the Makey exe.

Command: $(SolutionDir)Bin\Win_$(PlatformTarget)_$(Configuration)\$(ProjectName)\$(ProjectName).exe
Command Arguments: compiler=MSVC config=Release platform=Win_x64
Working Directory: $(SolutionDir)

• std::ofstream should probably not be used and std::stringstream need to be wrapped as feStringStream.
• Uses of std::ifstream should be replaced by something like feUWord File::readAllBytes(feStringView path, feUByte *&bytes).
• Instead of writing using std::ofstream, there should be a replacement function like void File::writeAllBytes(feStringView path, feUByte *&bytes, feUWord size). These might end up being annoying for text-only files so maybe wrappers like feString File::readAllText(feStringView path) and void File::writeAllText(feStringView path, feStringView text).

Switching to clang will net us cross-platform support

FeGen should generate serialization information for regular and opaque types

Standardize naming conventions while moving towards MVP

This is because the includes for a module aren't including the includes for its dependencies

As a code Dev, I only want to see the most relevant errors.

E.g. fail unit test, only show the unit test error. Not seemingly random ninja errors.

Right now the flow for FeMake is hard to follow after setup.

• AccumulateProjects should only iterate through the project directory and find files
• After all files are accumulated, build commands should be created
• After that, the ninja files should be written
• Remove any BuildCommand references from the MSVC stuff that's not related to the build alias

• Catch has a license available via github
• LLVM has a licence available on llvm.org
• ninja has a license available on github

Make sure to integrate x64 and x86 libs for them

Main things we need are:

• Good Intellisense support
• Build system integration
• Generatable project files

Remove Win_x64_Release executables from repository and add boostrap batch file

It'd be nice if we could ignore these merging issues.