Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Fix server-side fragmentation #1239

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Fix server-side fragmentation #1239

wants to merge 4 commits into from

Commits on Sep 12, 2020

  1. Fix server-side fragmentation 

    Old implementation attempted to implement fragmentation in the
TAO_OutputCDR inserters, but this does not work because array
and sequence types, including strings, were implemented at
the ACE_OutputCDR layer, which remained oblivious to fragmentation.
The old implementation also left TAO_OutputCDR with a dangerous
interface. The write_xxx function inherited from ACE_OutputCDR did
not fragment, only the inserters did.
The new implementation fragments in the low-level write_1, write_2,
write_4, write_8, write_16, and write_array functions, making these
virtual to ensure that constructed types like arrays and sequences
correctly fragment their elements, and similar for structs, etc.
    @j18ter
    j18ter committed Sep 12, 2020
    Configuration menu
    Copy the full SHA
    9c8f009 View commit details
    Browse the repository at this point in the history

  2. Removed trailing white space.

    @j18ter
    j18ter committed Sep 12, 2020
    Configuration menu
    Copy the full SHA
    a381ad7 View commit details
    Browse the repository at this point in the history

  3. Addressed TAO style checks.

    @j18ter
    j18ter committed Sep 12, 2020
    Configuration menu
    Copy the full SHA
    325c48a View commit details
    Browse the repository at this point in the history

  4. Removed trailing white space.

    @j18ter
    j18ter committed Sep 12, 2020
    Configuration menu
    Copy the full SHA
    1599a06 View commit details
    Browse the repository at this point in the history