src: cache invariant code motion

RafaelGSS · RafaelGSS · commit c457f9ed5a45 · 2024-08-05T11:15:41.000-03:00
PR-URL: #53879 Reviewed-By: Matteo Collina <matteo.collina@gmail.com> Reviewed-By: Paolo Insogna <paolo@cowtech.it> Reviewed-By: Tim Perry <pimterry@gmail.com> Reviewed-By: Yagiz Nizipli <yagiz@nizipli.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Marco Ippolito <marcoippolito54@gmail.com>
diff --git a/src/node_http2.cc b/src/node_http2.cc
@@ -1652,11 +1652,12 @@ void Http2Session::HandleSettingsFrame(const nghttp2_frame* frame) {
         int32_t settings_id = iv[i].settings_id;
         if (settings_id >=
             IDX_SETTINGS_COUNT) {  // unsupported, additional settings
+          auto iv_value = iv[i].value;
           for (size_t j = 0; j < numsettings; ++j) {
             if ((remote_custom_settings_.entries[j].settings_id & 0xFFFF) ==
                 settings_id) {
               remote_custom_settings_.entries[j].settings_id = settings_id;
-              remote_custom_settings_.entries[j].value = iv[i].value;
+              remote_custom_settings_.entries[j].value = iv_value;
               break;
             }
           }

Original file line number	Diff line number	Diff line change
`@@ -1652,11 +1652,12 @@ void Http2Session::HandleSettingsFrame(const nghttp2_frame* frame) {`
`1652`	`1652`	`int32_t settings_id = iv[i].settings_id;`
`1653`	`1653`	`if (settings_id >=`
`1654`	`1654`	`IDX_SETTINGS_COUNT) { // unsupported, additional settings`
	`1655`	`+ auto iv_value = iv[i].value;`
`1655`	`1656`	`for (size_t j = 0; j < numsettings; ++j) {`
`1656`	`1657`	`if ((remote_custom_settings_.entries[j].settings_id & 0xFFFF) ==`
`1657`	`1658`	`settings_id) {`
`1658`	`1659`	`remote_custom_settings_.entries[j].settings_id = settings_id;`
`1659`		`- remote_custom_settings_.entries[j].value = iv[i].value;`
	`1660`	`+ remote_custom_settings_.entries[j].value = iv_value;`
`1660`	`1661`	`break;`
`1661`	`1662`	`}`
`1662`	`1663`	`}`