Introduce command line argument to enable "unused warnings" about zserio HOT 5 CLOSED

The idea is good, but we should solve syntax and wrong usage:

Possible syntaxes:

1. root struct Root { ... };
2. struct root Root { ... };
3. main struct Root { ... };
4. struct main Root { ... };
5. rootstruct Root { ... };
6. mainstruct Root { ... };
7. root Root;
struct Root { ... };
8. main Root;
struct Root { ... };
9. struct Root^ { ... };

Wrong usage:

root struct MarkedRoot
{
    uint32 value;
};

struct Structure
{
    MarkedRoot markedRoot; // marked root is used, what to do? warning?!
}

from zserio.

Initial idea was to use something like

root struct Element1
{}

The above would not throw the warning for unused element. But the element itself could of course be reused by other structs. So basically there would be no "wrong usage" at all.

from zserio.

We think that the root keyword isn't consistent enough. For example, it seems very strange to have root const string ID = "Zserio", or root subtype int32 MySubtype. And also for users it might be difficult to keep in mind which entities can be "root" - e.g. sql_database or service are always "root".

Therefore we propose to introduce new command line option -> -withUnusedWarnings / -withoutUnusedWarnings(default). Then the user can just run the tool with -withUnusedWarnings to check which types are not used. In future, we could add something like "unusedWarningsBlackList" to suppress the warning for types which are supposed to be "root". This solution will not do mess in the language.

from zserio.

OK, I agree. So the default would be that the tool does not provide the unused warnings. Only if you explicitly turn it on. Sounds good!

from zserio.

yes, by default you get no unused warnings

from zserio.