Right now AllRequiredFieldsValid is called after LoadFromBuf and before SaveToBuf in the anchestor class.
If one tries to avoid required fields however (see https://developers.google.com/protocol-buffers/docs/proto#specifying-field-rules) and want's to implement it's own validation, it would be nice if the anchestor defined some sort of validation method and that the generator overrides this and calls it for all child messages which have a value. AllRequiredFieldsValid should be incorporated there as well, so that there's only one validation mechanism.
Right now AllRequiredFieldsValid is called after LoadFromBuf and before SaveToBuf in the anchestor class.
If one tries to avoid required fields however (see https://developers.google.com/protocol-buffers/docs/proto#specifying-field-rules) and want's to implement it's own validation, it would be nice if the anchestor defined some sort of validation method and that the generator overrides this and calls it for all child messages which have a value. AllRequiredFieldsValid should be incorporated there as well, so that there's only one validation mechanism.