Skip to content

Conversation

@Ykidia
Copy link
Contributor

@Ykidia Ykidia commented Jan 2, 2026

raw() = header() + body() + tail()
This is a real helper to make code a little cleaner, simpler and gives more freedom to choose the internal implementation of data storage in TreeItem.

@NikolajSchlej
Copy link
Collaborator

I do not like the word 'raw' to mean 'head+body+tail', as 'raw' means 'uncooked' or 'unprepared' or 'unprocessed', and here we have concatenation of up to three UByteArrays as a result of something that means 'nothing had been done with it just yet'.

'whole' maybe?

@Ykidia
Copy link
Contributor Author

Ykidia commented Jan 2, 2026

I do not like the word 'raw' to mean 'head+body+tail', as 'raw' means 'uncooked' or 'unprepared' or 'unprocessed', and here we have concatenation of up to three UByteArrays as a result of something that means 'nothing had been done with it just yet'.

'whole' maybe?

"raw" because of unstructured nature, in front of structured - header, body, tail, etc. Thought about this a lot and decided that "raw data" was the most appropriate name for me.
But if you dont't like this, we can use something like "entire", "total", "direct". But all these words imply the clarification "data", as it seems to me. But the "data" has a slightly different meaning in model classes...
Honestly, the word "whole" does not highlight the sense of integrated and, simultaneously, unstructured data, but acts as a filler word, in my opinion. But, maybe, "raw" also doesn't describe it completely.
Then we should try "entire" or something else ("total" and "direct" describes norm but sounds bad for me)?

@Ykidia
Copy link
Contributor Author

Ykidia commented Jan 2, 2026

New idea)! What about "blob" or "plain"?

@NikolajSchlej
Copy link
Collaborator

'entire' is alright

@Ykidia
Copy link
Contributor Author

Ykidia commented Jan 3, 2026

Done. Now entire() = header() + body() + tail().

@NikolajSchlej NikolajSchlej merged commit 152bfb1 into LongSoft:new_engine Jan 4, 2026
8 of 9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants