Copilot commented on code in PR #2988:
URL: https://github.com/apache/fory/pull/2988#discussion_r2587851935
##########
benchmarks/cpp_benchmark/README.md:
##########
@@ -59,16 +59,21 @@ cmake --build . -j$(nproc)
## Benchmark Cases
-| Benchmark | Description
|
-| -------------------------------- |
------------------------------------------------------------------- |
-| `BM_Fory_Struct_Serialize` | Serialize a simple struct with 8 int32
fields using Fory |
-| `BM_Protobuf_Struct_Serialize` | Serialize the same struct using Protobuf
|
-| `BM_Fory_Struct_Deserialize` | Deserialize a simple struct using Fory
|
-| `BM_Protobuf_Struct_Deserialize` | Deserialize the same struct using
Protobuf |
-| `BM_Fory_Sample_Serialize` | Serialize a complex object with various
types and arrays using Fory |
-| `BM_Protobuf_Sample_Serialize` | Serialize the same object using Protobuf
|
-| `BM_Fory_Sample_Deserialize` | Deserialize a complex object using Fory
|
-| `BM_Protobuf_Sample_Deserialize` | Deserialize the same object using
Protobuf |
+| Benchmark | Description
|
+| ------------------------------------- |
------------------------------------------------------------------- |
+| `BM_Fory_Struct_Serialize` | Serialize a simple struct with 8
int32 fields using Fory |
+| `BM_Protobuf_Struct_Serialize` | Serialize the same struct using
Protobuf |
+| `BM_Fory_Struct_Deserialize` | Deserialize a simple struct using
Fory |
+| `BM_Protobuf_Struct_Deserialize` | Deserialize the same struct using
Protobuf |
+| `BM_Fory_Sample_Serialize` | Serialize a complex object with
various types and arrays using Fory |
+| `BM_Protobuf_Sample_Serialize` | Serialize the same object using
Protobuf |
+| `BM_Fory_Sample_Deserialize` | Deserialize a complex object using
Fory |
+| `BM_Protobuf_Sample_Deserialize` | Deserialize the same object using
Protobuf |
+| `BM_Fory_MediaContent_Serialize` | Serialize a complex object with
Media and Images using Fory |
+| `BM_Protobuf_MediaContent_Serialize` | Serialize the same object using
Protobuf |
+| `BM_Fory_MediaContent_Deserialize` | Deserialize a complex object with
Media and Images using Fory |
+| `BM_Protobuf_MediaContent_Deserialize`| Deserialize the same object using
Protobuf |
Review Comment:
Missing space after the pipe character before "Deserialize". The table cell
separator should have proper spacing for consistency with other rows.
```suggestion
| `BM_Protobuf_MediaContent_Deserialize` | Deserialize the same object using
Protobuf |
```
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
