mirror of
https://chromium.googlesource.com/libyuv/libyuv
synced 2026-06-15 08:26:06 +08:00
8a564cacee
Patch Set 2:
(1 comment)
Patch-set: 2
Attention: {"person_ident":"Gerrit User 1001562 \u003c1001562@3ce6091f-6c88-37e8-8c75-72f92ae8dfba\u003e","operation":"REMOVE","reason":"\u003cGERRIT_ACCOUNT_1001562\u003e replied on the change"}
73 lines
2.2 KiB
Plaintext
73 lines
2.2 KiB
Plaintext
{
|
|
"comments": [
|
|
{
|
|
"unresolved": true,
|
|
"key": {
|
|
"uuid": "ab204c48_7178569c",
|
|
"filename": "/COMMIT_MSG",
|
|
"patchSetId": 2
|
|
},
|
|
"lineNbr": 11,
|
|
"author": {
|
|
"id": 1002219
|
|
},
|
|
"writtenOn": "2026-06-05T19:04:45Z",
|
|
"side": 1,
|
|
"message": "No `Bug:`?",
|
|
"revId": "91ca74f2e84cc1c45d5ba5bcf755899af81d45e5",
|
|
"serverId": "3ce6091f-6c88-37e8-8c75-72f92ae8dfba"
|
|
},
|
|
{
|
|
"unresolved": false,
|
|
"key": {
|
|
"uuid": "8743d517_44318757",
|
|
"filename": "/COMMIT_MSG",
|
|
"patchSetId": 2
|
|
},
|
|
"lineNbr": 11,
|
|
"author": {
|
|
"id": 1001562
|
|
},
|
|
"writtenOn": "2026-06-05T19:06:14Z",
|
|
"side": 1,
|
|
"message": "No. I am analyzing libyuv myself.",
|
|
"parentUuid": "ab204c48_7178569c",
|
|
"revId": "91ca74f2e84cc1c45d5ba5bcf755899af81d45e5",
|
|
"serverId": "3ce6091f-6c88-37e8-8c75-72f92ae8dfba"
|
|
},
|
|
{
|
|
"unresolved": false,
|
|
"key": {
|
|
"uuid": "66950250_7c62af7d",
|
|
"filename": "source/convert_to_argb.cc",
|
|
"patchSetId": 2
|
|
},
|
|
"lineNbr": 65,
|
|
"author": {
|
|
"id": 1001562
|
|
},
|
|
"writtenOn": "2026-06-05T18:45:57Z",
|
|
"side": 1,
|
|
"message": "Before we negate `src_height` and `crop_height`, we need to make sure they are not equal to `INT_MIN`. So I moved the validation of parameters to the beginning of the function.",
|
|
"revId": "91ca74f2e84cc1c45d5ba5bcf755899af81d45e5",
|
|
"serverId": "3ce6091f-6c88-37e8-8c75-72f92ae8dfba"
|
|
},
|
|
{
|
|
"unresolved": false,
|
|
"key": {
|
|
"uuid": "a56246da_72346154",
|
|
"filename": "source/convert_to_i420.cc",
|
|
"patchSetId": 2
|
|
},
|
|
"lineNbr": 52,
|
|
"author": {
|
|
"id": 1001562
|
|
},
|
|
"writtenOn": "2026-06-05T18:45:57Z",
|
|
"side": 1,
|
|
"message": "Before we negate `src_height` and `crop_height`, we need to make sure they are not equal to `INT_MIN`.\n\nThe validation of parameters uses the local variables `abs_src_height` and `abs_crop_height`, so it needs to be split into two parts.",
|
|
"revId": "91ca74f2e84cc1c45d5ba5bcf755899af81d45e5",
|
|
"serverId": "3ce6091f-6c88-37e8-8c75-72f92ae8dfba"
|
|
}
|
|
]
|
|
} |