-
Notifications
You must be signed in to change notification settings - Fork 1.4k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Removes Resource Flags and moves fields to DisableStruct #5945
Removes Resource Flags and moves fields to DisableStruct #5945
Conversation
I remember seeing end of word messages earlier. Did they get deleted in the merge with upcoming? |
I deleted it after #5946 because it looked weird. I can bring it back but I would argue that it would need a comment when the word starts and ends so it isn't mixed up with other fieds. |
Maybe instead of a "word start", a newline would be better? |
oh that's a good idea |
let me know if this is okay. to cut down on |
A word is 4 bytes (u32), not 1 byte |
Fixed |
Oh. I thought you were going to split it by word. |
will do |
Suggestion to revert the end of byte marks and do the reordering in a new pr. |
Sure 👍 |
ready for re-review |
Sorry, last master->upcoming merge added a conflict 🙏 |
solved |
The
ResourceFlags
struct was unreasonably convoluted. I've moved everything to theDisbaleStruct
and removed it so it isn't used anymore.