1. Let CommHookInterface::processFuture be a non-virtual method, which can be shared by both Python and C++ implementations.
2. Merge 1-line PythonCommHook constructor into the header for simplicity.
3. Rename processFuture method as parseFromHookResult for readability.
4. Simplify the comments.

Original PR issue: C++ DDP Communication Hook #46348

Differential Revision: [D24374282](https://our.internmc.facebook.com/intern/diff/D24374282/)

[ghstack-poisoned]

…onCommHook."

1. Let CommHookInterface::processFuture be a non-virtual method, which can be shared by both Python and C++ implementations.
2. Merge 1-line PythonCommHook constructor into the header for simplicity.
3. Rename processFuture method as parseFromHookResult for readability.
4. Simplify the comments.

Original PR issue: C++ DDP Communication Hook #46348

Differential Revision: [D24374282](https://our.internmc.facebook.com/intern/diff/D24374282/)

[ghstack-poisoned]

…onCommHook."

1. Let CommHookInterface::processFuture be a non-virtual method, which can be shared by both Python and C++ implementations.
2. Merge 1-line PythonCommHook constructor into the header for simplicity.
3. Rename processFuture method as parseFromHookResult for readability.
4. Simplify the comments.

Original PR issue: C++ DDP Communication Hook #46348

Differential Revision: [D24374282](https://our.internmc.facebook.com/intern/diff/D24374282/)

[ghstack-poisoned]

…onCommHook."


1. Merge 1-line PythonCommHook constructor into the header for simplicity.
2. Move the implementation of PythonCommHook destructor from the header file to the cpp file.
3. Rename processFuture method as parseHookResult for readability.
4. Simplify some comments.

Original PR issue: C++ DDP Communication Hook #46348

Differential Revision: [D24374282](https://our.internmc.facebook.com/intern/diff/D24374282/)

[ghstack-poisoned]

…onCommHook."


1. Merge 1-line PythonCommHook constructor into the header for simplicity.
2. Move the implementation of PythonCommHook destructor from the header file to the cpp file.
3. Rename processFuture method as parseHookResult for readability.
4. Simplify some comments.

Original PR issue: C++ DDP Communication Hook #46348

Differential Revision: [D24374282](https://our.internmc.facebook.com/intern/diff/D24374282/)

[ghstack-poisoned]

…onCommHook."


1. Merge 1-line PythonCommHook constructor into the header for simplicity.
2. Move the implementation of PythonCommHook destructor from the header file to the cpp file.
3. Rename processFuture method as parseHookResult for readability.
4. Simplify some comments.

Original PR issue: C++ DDP Communication Hook #46348

Differential Revision: [D24374282](https://our.internmc.facebook.com/intern/diff/D24374282/)

[ghstack-poisoned]

…onCommHook."


1. Merge 1-line PythonCommHook constructor into the header for simplicity.
2. Rename processFuture method as parseHookResult for readability.
3. Simplify some comments.

Original PR issue: C++ DDP Communication Hook #46348

Differential Revision: [D24374282](https://our.internmc.facebook.com/intern/diff/D24374282/)

[ghstack-poisoned]

…onCommHook."


1. Merge 1-line PythonCommHook constructor into the header for simplicity.
2. Rename processFuture method as parseHookResult for readability.
3. Simplify some comments.

Original PR issue: C++ DDP Communication Hook #46348

Differential Revision: [D24374282](https://our.internmc.facebook.com/intern/diff/D24374282/)

[ghstack-poisoned]

…onCommHook."


1. Merge 1-line PythonCommHook constructor into the header for simplicity.
2. Rename processFuture method as parseHookResult for readability.
3. Simplify some comments.

Original PR issue: C++ DDP Communication Hook #46348

Differential Revision: [D24374282](https://our.internmc.facebook.com/intern/diff/D24374282/)

[ghstack-poisoned]

…onCommHook."


1. Merge 1-line PythonCommHook constructor into the header for simplicity.
2. Rename processFuture method as parseHookResult for readability.
3. Simplify some comments.

Original PR issue: C++ DDP Communication Hook #46348

Differential Revision: [D24374282](https://our.internmc.facebook.com/intern/diff/D24374282/)

[ghstack-poisoned]

…onCommHook."


1. Merge 1-line PythonCommHook constructor into the header for simplicity.
2. Rename processFuture method as parseHookResult for readability.
3. Simplify some comments.

Original PR issue: C++ DDP Communication Hook #46348

Differential Revision: [D24374282](https://our.internmc.facebook.com/intern/diff/D24374282/)

[ghstack-poisoned]

…onCommHook."


1. Merge 1-line PythonCommHook constructor into the header for simplicity.
2. Move the implementation of PythonCommHook destructor from the header file to cpp file.
3. Rename processFuture method as parseHookResult for readability.
4. Simplify some comments.

Original PR issue: C++ DDP Communication Hook #46348

Differential Revision: [D24374282](https://our.internmc.facebook.com/intern/diff/D24374282/)

[ghstack-poisoned]