Github user PivotalSarge commented on a diff in the pull request:

    https://github.com/apache/geode-native/pull/75#discussion_r108496561
  
    --- Diff: 
docs/geode-native-docs/dotnet-caching-api/implementing-igfserializable.html.md.erb
 ---
    @@ -31,21 +31,21 @@ Examples follow the procedure.
         void ToData(DataOutput output)
         ```
     
    -    The `ToData` function is responsible for copying all of the data 
fields for the object to the object stream. The `DataOutput` class represents 
the output stream and provides methods for writing the primitives in a network 
byte order. For details, see the API documentation for `DataOutput` at 
[http://gemfire-apis.docs.pivotal.io](http://gemfire-apis.docs.pivotal.io).
    +    The `ToData` function is responsible for copying all of the data 
fields for the object to the object stream. The `DataOutput` class represents 
the output stream and provides methods for writing the primitives in a network 
byte order.
     
     2.  Implement the `FromData` function that consumes a data input stream 
and repopulates the data fields for the object:
     
         ``` pre
         void fromData (DataInput& input)
         ```
     
    -    The `DataInput` class represents the input stream and provides methods 
for reading input elements. The `FromData` function must read the elements of 
the input stream in the same order that they were written by `ToData`. For more 
about this, see the API documentation for `DataInput` at 
[http://gemfire-apis.docs.pivotal.io](http://gemfire-apis.docs.pivotal.io).
    +    The `DataInput` class represents the input stream and provides methods 
for reading input elements. The `FromData` function must read the elements of 
the input stream in the same order that they were written by `ToData`.
     
     3.  Implement the `ClassId` function to return an integer which is unique 
for your class (in the set of all of your user-defined classes).
     
     ## Simple BankAccount Class
     
    -This example shows a simple class, `BankAccount`, that encapsulates two 
`ints`: `customerId` and `accountId`:
    +This example shows a simple class, `BankAccount`, that encapsulates the 
two `ints` `customerId` and `accountId`:
    --- End diff --
    
    Maybe a comma between `ints` and `customerId`?


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

Reply via email to