For as far as our own (not third-party) code is concerned, we should investigate and where possible solve compiler warnings emitted by the build process.
Fixes for the remaining -Wclass-memaccess (GCC8) warnings in /js are not trivial and have (so far) only caused build bustage in my attempts to port additional fixes the past few days. I am likely going to have to leave fixing these to someone more knowledgeable with the code in /js.
Also posting a list of current -Wunused-but-set-variable, -Wunused-variable, & -Wunused-function warnings displayed by GCC 7.3 for reference: https://gist.github.com/trav90/19cf4a2de35ac794f382b780074ea2e4
Deleting a branch is permanent. It CANNOT be undone. Continue?