Small changes and improvements

[qtdyn]

This PR adds/fixes the following:

• Add .gitignore file
  • Currently only ignores bin/ to prevent pushing built objs/binaries
• Add proper Makefile
  • This allows for caching of .o files, meaning files will only be recompiled if they have changed
  • Added variables like CFLAGS and COPTS for compilation and optimization flags
• Fixed warnings related to the introduction of more warning flags in CFLAGS
• Added some sort of verification for file reading/writing in the input
  • You shouldn't be allowed to pass an input file you can't read or write to somewhere you don't have access to
• If the output path isn't specifically passed, try and guess it
  • This might be a bit janky? I don't really see how but my gut tells me this is horrible code lol

Note: The Makefile model is based on my school's Makefiles, so it might be a bit opinionated.

Let me know if anything needs changes and/or total removal and I'll be happy to do so :)

[KMatias123]

Being inconsistent with the indentation is bad since people have different tab widths in their editors but the space width is always the same, so the files will look weird to people with for example 2-wide tabs.

Returning 0 after failing will mostly be a problem if another program is executing construct, as it means EXIT_SUCCESS and the other program will think that construct finished successfully and will continue executing.

[qtdyn]

Apparently vim didnt catch on the spaces, whoops. I'll go back to it in a day or two to fix that, thanks :)

[qtdyn]

Did some quick merges on the web editor, probably doesn't compile unless i'm lucky lol

[qtdyn]

(for the record, this compiles now) 👍

[Thomas-995]

Currently busy with quarterly exams, will look at this soon (week or two), thanks for contributing!

[qtdyn]

Good luck on your exams and sorry if that sounded spam-ish ^^'

[KMatias123]

Forgot to finish this xD

[immosfet]

Why is my account inserted in between you guys' conversations? I don't know any of you guys Em seg., 30 de dez. de 2024 às 18:09, matias ***@***.***> escreveu:

…

***@***.**** commented on this pull request. Forgot to finish this xD ------------------------------ In src/construct.cpp <#1 (comment)> : > } std::ifstream inpfile(inputfile); + if (!inpfile.is_open()) { + std::cerr << "Could not open input file" << std::endl; + return 1; + } + if (!inpfile.good()) { + std::cerr << "Input file is not good" << std::endl; This could use a more descriptive ⬇️ Suggested change - std::cerr << "Input file is not good" << std::endl; + std::cerr << "Error reading input file" << std::endl; Maybe print out the error? — Reply to this email directly, view it on GitHub <#1 (review)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/BFZPRNLOXHZGZTOLFDR44IT2IGY63AVCNFSM6AAAAABTSMWFHKVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMZDKMBUGU3DIMZWG4> . You are receiving this because you are subscribed to this thread.Message ID: ***@***.***>