Use new TQ_METHOD, TQ_SIGNAL, TQ_SLOT defines #20
Merged
MicheleC
merged 1 commits from replace/signal-n-slot into master 4 months ago
SlavekB
Loading…
Reference in new issue
There is no content yet.
Delete Branch 'replace/signal-n-slot'
Deleting a branch is permanent. It CANNOT be undone. Continue?
As per title.
c0acd68573to0dc42f763d4 months ago0dc42f763dtoa2b25cd3934 months agoa2b25cd393to209ac1e5614 months agoThere are several cases where
METHODare not TQt methods. Of course it's just part of the comment.// NMETHODS field contains the number of method identifier octets that// appear in the METHODS field.// The values currently defined for TQT_METHOD are:// The values currently defined for METHOD are:These are not TQt methods.
I believe you were reviewing the PR before my second update (commit
209ac1e561). These wrongMETHODrenames have already been fixed. Fyi, I usually push the PR first and the review, because TGW highlights the changes much better than git does.Now it looks fine. Maybe there was just a coincidence in the time of time.
{// The server selects from one of the methods given in METHODS, and// sends a TQT_METHOD selection message:// sends a METHOD selection message:These are not TQt methods.
//// +----+--------+// |VER | TQT_METHOD |// |VER | METHOD |These are not TQt methods.
// +----+--------+//// If the selected TQT_METHOD is X'FF', none of the methods listed by the// If the selected METHOD is X'FF', none of the methods listed by theThese are not TQt methods.
// client are acceptable, and the client MUST close the connection.//// The values currently defined for TQT_METHOD are:// The values currently defined for METHOD are:These are not TQt methods.
All seems good.
209ac1e561into master 4 months agoReviewers
209ac1e561.