forked from public/pysim
Patch Set 2:
(1 comment)
Patch-set: 2
Attention: {"person_ident":"Gerrit User 1000028 \u003c1000028@035e6965-6537-41bd-912c-053f3cf69326\u003e","operation":"REMOVE","reason":"\u003cGERRIT_ACCOUNT_1000028\u003e replied on the change"}
21 lines
737 B
Plaintext
21 lines
737 B
Plaintext
{
|
|
"comments": [
|
|
{
|
|
"unresolved": true,
|
|
"key": {
|
|
"uuid": "1f6121ac_47ab3cb6",
|
|
"filename": "pySim/transport/__init__.py",
|
|
"patchSetId": 2
|
|
},
|
|
"lineNbr": 328,
|
|
"author": {
|
|
"id": 1000028
|
|
},
|
|
"writtenOn": "2026-03-09T16:03:21Z",
|
|
"side": 1,
|
|
"message": "What confuses me here is that the spec also mentions a 9xxx (but not 9000, and presumably also 9100 as a candidate for this). Technically this seems not to make sense and it also breaks the unit-tests. (for example: 9404, which means FID not found, would also end u here.)",
|
|
"revId": "22ec9a708ab4128df9f4dd340df2e263f9ebd539",
|
|
"serverId": "035e6965-6537-41bd-912c-053f3cf69326"
|
|
}
|
|
]
|
|
} |