Fix tmux list-session parsing error by removing buffer_sample field from fetch_objs function#473
Fix tmux list-session parsing error by removing buffer_sample field from fetch_objs function#473hongwen000 wants to merge 1 commit intotmux-python:masterfrom
Conversation
…rom fetch_objs function
|
|
|
@hongwen000 Thank you for this contribution and the issues There's another possibility where we can possibly keep the separator: We could change the separator from If you try |
Codecov Report
@@ Coverage Diff @@
## master #473 +/- ##
==========================================
- Coverage 88.97% 88.91% -0.07%
==========================================
Files 35 35
Lines 3484 3483 -1
Branches 484 484
==========================================
- Hits 3100 3097 -3
- Misses 276 278 +2
Partials 108 108
8000 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
|
@tony Yes, this modification can also solve my problem. Changing the delimiter to a rare Unicode character is a nice idea. Although the this character existing in buffer_sample is theoretically possible, but it is almost unlikely. One "safe" solution is to call tmux list-session twice, to get buffer_sample and other fields separately. But this would increase additional complexity. In any case, I agree to change the delimiter to ␞ to solve this problem. |
|
@hongwen000 I am going to release #475, come back, and see if it alleives the issue for you. |
See issue #472 and issue #471