Packet generator doesn't handle cycles

[L3nn0x]

The generator detects cycles, but doesn't do anything about it.

[RavenX8]

what's a cycle?

[L3nn0x]

the xml is a directed graph so technically we can have cycles (a type that uses another type, which itself uses the first type). It'd require a forward declaration of the second type probably, and use a unique_ptr<> instead of a variable on the stack to break the dependency cycle.

[RavenX8]

oh cool. that would be nice.

[L3nn0x]

Not sure if it'd be useful tho, so I won't change it until we need it.