25bec4c587
lib/nbt: Validator: Fix bytes not being added up correctly in multiple places
2022-08-15 10:51:50 +02:00
589cf1ddaf
lib/nbt: NBT validator: Fix wrong function declaration in the header, fix not using the currentPosition variable when accessing data
2022-08-15 09:53:06 +02:00
884a5239c6
lib/nbt: fix a bug in NBT::helper::readString() which caused it to asuume that dataSize is the size of the string
2022-08-15 09:51:46 +02:00
9190cad80d
lib/nbt: finish implementation of validateRawNBTData() and fix a critical macro-induced bug
...
I did a `#define return` and then tried to `if () return;` everywhere...
2022-08-15 08:50:07 +02:00
a862590370
lib/nbt: Start implementing the NBT validator
...
In theory, this is it. It’s just missing the portion that deals with lists
and unit tests. Both will each likely require similar effort to this.
2022-08-15 05:20:05 +02:00
3995e97f03
lib/javacompat: Make the endianness error message refer to the correct function
2022-08-15 02:07:00 +02:00
c9ec524db1
test/nbt_size_helpers: Implement tests for valid input
2022-08-13 17:32:47 +02:00
73ae58e522
test/assert: Add line number to assertion failed message
2022-08-13 17:32:47 +02:00
acc19ae100
lib/nbt: Change behavior of totalTagSize to treat encounters of compounds and lists as errors
...
I stumbled over this when writing the unit test. Previously, it would return
an error code but explicitly mark it as not being an error. This was intended
behavior but I decided to change it because I didn’t anticipate it when writing
the test.
Technically `ErrorOr<T>` can be used to pass any message alongside `T`,
but practically, it is reasonable to assume that the error code is
`ErrorCodes::SUCCESS` when `isError` is false. Therefore, this feature
should be really only used in the weirdest edge cases - if at all.
Even then, it is most likely still preferable to flag it as an error and
just hand the resulting `T` back using the long constructor
`ErrorOr<T>(bool, uint8_t, T)`.
2022-08-13 17:32:47 +02:00
149285c357
lib/nbt: Finish implementing containedDataLength, rename nextTagTotalSize->totalTagSize and nextTagDataLength->containedDataLength
2022-08-13 17:32:47 +02:00
0c92bdf8fd
test/nbt_size_helpers: begin adding unit tests for lib/nbt's new nextTag size helpers
2022-08-13 17:32:47 +02:00
86f1ef596f
lib/nbt: Begin implementing nextTagDataLength
2022-08-13 17:32:47 +02:00
027f324f03
lib/nbt: Fix a bug in nextTagTotalSize and significantly improve readability by removing redundant code
2022-08-13 17:32:47 +02:00
f5d85da98c
lib/nbt: Move the functions for getting tag sizes into the helper namespace, give up on handling lists the same as all other tags
...
I tried dealing with lists in the same way as with other more basic tags
but came to the conclusion that this is most likely not feasible in the same
way that it is not feasible for compounds. It would require a mini-parser
that can deal with all sorts of tags (including nested lists and compounds).
Instead, an approach more similar to the recursion for compound tags will
be used (using its own function to deal with the missing tag headers ofc).
2022-08-13 17:32:47 +02:00
396b9673fd
lib/nbt: Various minor fixes to get the program to compile properly
2022-08-13 17:32:47 +02:00
68fbf3ae20
lib/nbt: remove a function used to get the next tag type which introduced unnecessary complexity
2022-08-13 17:32:47 +02:00
5400790e78
test/nbt*: rename files, move byte tag object test from helper test file into its own file
2022-08-13 17:32:47 +02:00
c7dd5471dd
lib/nbt: Start implementing NBT validator
2022-08-13 17:32:47 +02:00
8048dc8891
tools/hexnet: Put the new usage generator to use and remove prefixes for IPv4 and IPv6
...
The prefixes were part of a planned feature but since a connection is
either IPv4 or IPv6 but never both, it would have been completely useless
to specify which to use. Instead, only TCP and UDP will need to be specified.
2022-08-12 12:30:55 +02:00
a1f16e6f6b
lib/cli: Fix the usage text generator not dealing well with absent sections
2022-08-12 11:59:42 +02:00
cb7b5ddba7
lib/cli: Add usage generator.
2022-08-12 13:35:56 +05:30
e0648720bb
tools/hexnet: Implement ipv6 support.
2022-08-11 22:55:12 +05:30
ebcf436a18
lib/cli: delete duplicate header file.
2022-08-11 18:57:22 +05:30
aef91fe7cd
test/nbt_helpers: Fix wrong test pass message
2022-08-04 07:50:20 +02:00
4af9003761
Code style: I just decided to accept that float and double exist and that we can just assume they are 32 and 64 bits repectively.
...
This isn't going to run on an Arduino or anything like that anyway.
2022-08-04 07:47:24 +02:00
704b440d5a
lib/nbt: Add tag classes
2022-08-04 00:01:12 +05:30
608767f5c2
tools/hexnet: Add more command line flags and options
...
This only adds the options to the parser. They aren't used anywhere in the code yet.
2022-08-02 03:42:37 +02:00
e31bff0802
test/nbt_writestring_failure_mode: Add license information
2022-08-02 03:37:10 +02:00
5c73308934
Rename all headers from .h++ to .hpp
...
Idk why I did that in the first place. Probably bc hpp looks stupid.
But having a + in a file name bugs me just as much. And other ppl as well.
So I changed it.
2022-08-02 03:35:08 +02:00
b59fe1857e
lib/cli: minor refactoring to make things less confusing and nicer to use
...
I renamed "unpositional arguments" to "options" and "positional arguments" to "arguments".
This is intended to make the code more readable and easier to type out.
2022-08-02 03:16:54 +02:00
69f15e928a
lib/cli: Add fields for a short description and additional usage information to the arguments parser
...
This is in preparation for building the help text generator.
2022-08-02 02:03:50 +02:00
d0d02fc8d2
tools/hexnet: Move the TCP reading portion to a thread
...
This was too straight-forward to not just do it when I previously worked
on hexnet. Why didn't I just do it? Idk.
2022-08-02 01:07:20 +02:00
1308327fae
tests: OCD fixes lol
...
sorry
2022-08-02 01:04:45 +02:00
4582c3e595
test/nbt_helpers: move the test that aborts to its own program
2022-08-02 00:41:11 +02:00
28719072bb
fix a compiler warning
2022-08-01 16:39:18 +02:00
4f1ad714bd
lib/nbt.cpp: Implement writeString function
2022-07-28 17:15:04 +05:30
adc9a7f36b
tools/hexnet: prepare for multithreading
...
Multithreading will be needed to simultaneously receive and send data.
The preparations include:
- move all the settings of the program into global scope
- add mutexes
- move the code that reads from the TCP socket into a dedicated function
2022-07-25 15:55:40 +02:00
b044503951
test/cli_argument_parser: get rid of useless parameters
...
(pointed out by compiler warning)
2022-07-23 10:55:03 +02:00
6baff11ebd
test/cli_argument_parser: implement tests for generated error messages
2022-07-21 09:41:04 +02:00
c9d6cf0b5e
lib/cli: minor consistency changes
2022-07-21 09:39:45 +02:00
a51c65c9f2
test/javacompat: get rid of a compiler warning
2022-07-20 19:22:06 +02:00
0dcc579bb5
test/cli_arguments_parser: extend some tests, implement all remaining parser tests except checking for error messages
2022-07-20 19:16:02 +02:00
db3b133f88
lib/cli: fix trailing incomplete unpositional arguments
...
The parser used to rely on the next iteration of the loop to detect
if an unpositional argument was missing its value, this has now been
fixed by adding an additional check on unpositional arguments waiting
for a value that detects if the end of the loop has been reached
2022-07-20 18:30:25 +02:00
2cc2543b2a
lib/error: fix compiler error with g++11 or newer
...
This wasprobably a syntax error on my part.
2022-07-20 09:58:58 +02:00
a0d2974f0a
test/cli_argument_parser: fix programmer-generated segfault
2022-07-20 08:46:06 +02:00
4659946a2f
src/test: free pointers
2022-07-20 12:13:47 +05:30
b5312aeb58
lib/nbt: implement readString NBT helper function
2022-07-20 12:08:04 +05:30
9562ae7be9
Merge branch 'master' of https://lostcave.ddnss.de/git/BodgeMaster/FOSS-VG
2022-07-20 08:07:03 +02:00
b1733bc007
tools/hexnet: initial implementation of TCP receive only server
2022-07-20 08:06:45 +02:00
d315c6fcfc
Merge branch 'broken'
2022-07-20 10:39:16 +05:30
2f38636a27
lib/javacompat: implement exportJavaString
2022-07-20 10:38:55 +05:30
0462756451
lib: rename java_string to javacompat
2022-07-20 08:44:56 +05:30
c1b6c3005f
lib/java_string: fix end character issues
2022-07-20 08:39:29 +05:30
45538e156a
lib/java_string: replace std::vector with std::string
2022-07-20 04:23:06 +02:00
34e30c0bd4
test/java_string: replace tabs with spaces, fix how tiny_utf8::string is being instantiated
2022-07-20 04:02:44 +02:00
e184acde00
lib/java_string: rename functions
...
JavaCompat::importJavaFormatToString() → JavaCompat::importJavaString()
JavaCompat::exportStringToJavaFormat() → JavaCompat::exportJavaString()
2022-07-20 04:00:17 +02:00
09e2030a55
test/cli_argument_parser: More thorough unknwon flag test, add unknown unpositional argument test
2022-07-18 21:26:37 +02:00
42336c560a
Birthed an abomination, what the fuck
2022-07-18 12:13:10 +05:30
79f58a6242
lib/java_string: add placeholder functions
2022-07-17 13:02:45 +02:00
2cbf048f2e
lib/cli: make flags return more accurate error codes
...
revealed by writing a test and it behaving differently from what I expected
2022-07-17 12:09:19 +02:00
4ceaca745b
test/cli_argument_parser: notes outlining what additional tests are needed
2022-07-15 19:47:07 +02:00
db6e543c5c
test/cli_argument_parser: add a test for dealing with "empty" command lines
2022-07-15 19:46:20 +02:00
8835b90398
test/cli_argument_parser: rename everything to have nice long descriptive names
...
This is in preparation for adding more tests that require the same kinds of components.
2022-07-15 19:43:17 +02:00
efa4fa560e
tools/hexnet: replace built-in argument parser with lib/cli
...
This is the entire reason I build lib/cli in the first place:
Parsing arguments directly in a given program tends to get really messy,
not to mention all the different ways that arguments can be specified
and the redundant work required to do that on multiple programs.
2022-07-15 13:51:19 +02:00
4c3d2fdbbf
lib/cli: ArgumentsParser constructor can accept both char** and const char**
2022-07-15 13:49:43 +02:00
3b1d288d1e
test/cli_argument_parser add an assertion to the valid input test that expects the value of unpositional arguments that are not present to be ""
2022-07-15 13:44:56 +02:00
b18a97713d
test/cli_argument_parser: fix using delete instead of delete[] on pointer array
2022-07-15 11:47:19 +02:00
9ce979aa7c
lib/cli: first unit test implemented
2022-07-15 11:33:02 +02:00
82d611b984
lib/cli: various fixes
...
- argumentWaitingForValue was never reset after the value had been retrieved
- made it so the ArgumentsParser constructor can accept const char**
- correct usage of substr() and actually use the correct position in the first place
- fix a typo where I tried to dereference `this` with `-` instead of `->`
2022-07-15 11:21:18 +02:00
f5b0b74f94
lib/cli: remove useless arguments from getters, remove useless getter getProgramName (made the string public instead)
2022-07-15 09:11:39 +02:00
a1223ea4b9
Minor improvements and fixes.
...
test/nbt_helpers: add a headline to output
Build system: remove comma from array that I added bc I thought it was correct sytax, it wasn't
lib/cli: change argument order of PositionalArgument constructor to fit in with Flag and UnpositionalArgument
lib/cli: properly define what is private and what is public in the ArgumentsParser class
2022-07-15 08:54:09 +02:00
390087fc35
lib/cli: finish initial implementation
...
This should be all functionality that is needed for now.
The next step is building unit tests to check that everything works correctly.
Who am I kidding? There will most certainly be bugs in there given that this
code has been typed out without ever being run.
2022-07-14 04:57:48 +02:00
a2084e296a
lib/error: Remove ErrorOr<T>(bool) constructor
...
This allows ErrorOr<T> to be used for bool, as there would be conflicting
constructors otherwise.
Also, we have ErrorCodes::UNKNOWN now so, instead of `ErrorOr<T>(true)`,
`ErrorOr<T>(true, ErrorCodes::UNKNOWN)` can be used.
2022-07-14 03:58:13 +02:00
e6f4884b60
lib/error: finally removed that useless cpp file
2022-07-14 03:16:58 +02:00
91f3dfaa5e
lib/cli: Add a library for command line arguments parsing
...
This has the parser itself already implemented, all other functionality is still
missing. So are tests.
Yes, I’m making my own getopt. Let me.
2022-07-14 03:13:48 +02:00
7e049fcfd1
lib/error: rename and add error constants
2022-07-13 22:01:18 +02:00
c406555e4e
tools/hexnet: argument parser fix
2022-07-13 20:35:33 +02:00
5cb1a52ec4
tools/hexnet: replace "server" with "listen" because that's what netcat calls it and the argument we use is -l like netcat's listen mode
2022-07-13 20:30:27 +02:00
9639ef065f
tools/hexnet: Work on argument parser to finish suppurt for client mode
2022-07-13 20:27:58 +02:00
310011a6da
tools/hexnet: replace exit codes with macros to improve readability
2022-07-13 06:06:09 +02:00
6a5096dfa3
tools/hexnet: add server/client concept to arguments parser
2022-07-13 06:03:41 +02:00
b3cd8709fb
Tools: Add WIP hexnet tool. This will be similar to hexdump and netcat in one.
...
Yes, I am using this as an exercise for myself to figure out networking.
2022-07-12 05:13:01 +02:00
68484c6a20
NBT: Moar nbt helper functions. Add all functions in issue #18 except writeString
2022-07-06 18:28:02 +05:30
847b73c6ae
NBT: implement the overloaded writeInt8Array
2022-07-06 17:16:47 +05:30
8d8c1e6b90
NBT: change "[]" to "Array" in unit test messages.
2022-07-06 16:28:08 +05:30
b0ccc74409
NBT: Added writeInt8Array.
2022-07-06 16:27:32 +05:30
762ba7f089
NBT: add write helpers for float types
2022-07-04 19:43:31 +02:00
8c7be385ae
NBT: add write helpers for int32 and int64
2022-07-04 18:29:38 +02:00
6d62d995df
NBT: implement write helpers for int8 and int16
2022-07-04 18:03:40 +02:00
044593e081
NBT: add compiler errors to the endian specific sections
2022-07-02 16:41:54 +02:00
664632d111
NBT: add stub write helper functions
2022-07-02 02:08:32 +02:00
4f9577eb36
NBT: implement readFloat helpers
2022-07-01 21:15:18 +02:00
975cdd309d
NBT: implement NBT::helper::readInt64Array
2022-06-30 11:02:30 +02:00
edcf40d5a5
NBT: implement NBT::helper::readInt32Array
2022-06-30 10:45:12 +02:00
562fbcecbb
remove pointless placeholder, rename /test to /test_data because the actual tests live in /src/test
2022-06-29 12:34:27 +02:00
c87c519bb5
NBT: implement NBT::helper::readInt8Array
2022-06-29 12:22:54 +02:00
71fab21dfb
Error: add OVERRUN_ERROR
2022-06-29 11:32:33 +02:00
d43ef67ed1
NBT: fix bug in readInt64, improve readability
2022-06-29 00:05:02 +02:00
ed429633e5
NBT Test: fix begin of data test for int64, remove irrelevant (and architecture-specific) suffixes
2022-06-29 00:02:51 +02:00
7c12a92b2a
NBT: implement the integer helper functions
2022-06-28 22:04:04 +02:00
BodgeMaster