Skip to content

parse_journal_entry_log2: Remove unnecessary allocation#109

Merged
j4r0u53k merged 1 commit into
masterfrom
optimize-journalrw
Jun 9, 2026
Merged

parse_journal_entry_log2: Remove unnecessary allocation#109
j4r0u53k merged 1 commit into
masterfrom
optimize-journalrw

Conversation

@syyyr

@syyyr syyyr commented Jun 9, 2026

Copy link
Copy Markdown
Contributor

No description provided.

@syyyr syyyr requested a review from j4r0u53k June 9, 2026 10:40
@syyyr syyyr force-pushed the optimize-journalrw branch from db9bde5 to db8ab73 Compare June 9, 2026 10:40
@github-actions

github-actions Bot commented Jun 9, 2026

Copy link
Copy Markdown

Code coverage report for db8ab73

Coverage difference

✅ All new changes were covered.

Filename Stmts Miss Cover Missing
src/journalrw.rs -1 0 -0.02%
TOTAL -1 0 -0.01%

Overall coverage

Click to expand
Filename Stmts Miss Cover Missing
src/journalrw.rs 726 110 84.85% 76, 100, 132-137, 172, 303, 308, 314, 316, 331, 339-341, 357, 364, 374, 389, 397-400, 426-434, 447, 475-477, 491-528, 550-555, 559, 562, 576, 579-583, 588, 592-593, 598-599, 603-604, 608-609, 630-632, 644, 652-653, 675-701, 711-712, 716-717, 721-722, 726-727, 731-732, 736-737, 741-742, 746-747, 757-758, 762, 767-768, 964
src/streamrw.rs 110 19 82.73% 32-35, 70-74, 80, 113-115, 135-145
src/framerw.rs 141 37 73.76% 23, 33, 40-56, 81-87, 112-116, 123, 134, 150, 157-179, 201-203, 233
src/serialrw.rs 203 27 86.70% 44-47, 124-127, 131-138, 156-157, 203-205, 243-246, 261, 285-287
src/typeinfo.rs 846 299 64.66% 72-74, 103-105, 113, 120-121, 134, 138, 146, 159-244, 264, 272, 287-289, 293-295, 320-335, 350-351, 368-369, 380-385, 391, 399, 433-434, 455-461, 506-519, 584-603, 628-630, 657-668, 694-695, 706-735, 765-767, 788-790, 804-829, 837, 932-934, 941, 951, 961-963, 971-977, 997-1040, 1061-1069, 1081-1090, 1102, 1125-1127, 1144, 1181, 1184, 1189-1190, 1227-1228, 1231-1232, 1239, 1259-1261, 1265-1269, 1273-1275
src/rpcmessage.rs 462 202 56.28% 56, 69, 84-95, 107, 112-113, 118-123, 154-158, 162-165, 168-177, 181-186, 190, 196-198, 202, 227-228, 235, 239, 245-257, 292-297, 302-304, 308-310, 320-325, 339-344, 354, 380-385, 433-476, 489, 504-523, 541-556, 566-638
src/metamethod.rs 152 100 34.21% 26-34, 49, 51, 59-61, 88-91, 97, 99-105, 110-112, 143, 155-256, 288-290
src/client.rs 182 165 9.34% 24-271
src/datachange.rs 113 6 94.69% 100-104, 109
src/util.rs 289 52 82.01% 12-22, 31, 86-113, 150, 153, 184-194, 235-237
src/websocketrw.rs 67 24 64.18% 28-42, 55-64, 105-115, 128, 161-163
src/rpc.rs 208 60 71.15% 33-35, 39-76, 92-94, 118-124, 138, 152, 172-188, 195, 203-213, 225-227
src/rpcframe.rs 127 37 70.87% 25-30, 63, 72, 75, 78, 88-92, 120, 133-136, 152-183
src/rpcdiscovery.rs 262 48 81.68% 45-62, 72, 103-105, 166, 168-169, 179, 183-186, 211, 213-214, 221-232, 245-262, 272, 278, 283
src/canrw.rs 569 168 70.47% 34-58, 101-158, 176-203, 219-306, 331, 359-366, 395-398, 405-406, 416-418, 430-441, 476, 528-548
TOTAL 4457 1354 69.62%

HTML reports

@j4r0u53k j4r0u53k merged commit f6d84d0 into master Jun 9, 2026
10 checks passed
@j4r0u53k j4r0u53k deleted the optimize-journalrw branch June 9, 2026 12:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants