perf(skymp5-server): skip ACTI and FURN objects in AttachSaveStorage

[Pospelove]

Important

Add condition in AttachSaveStorage to skip ACTI and FURN objects in PartOne.cpp.

• Behavior:
  • In AttachSaveStorage in PartOne.cpp, added condition to skip objects of type ACTI and FURN when processing FF objects.
  • Logs a message when skipping these objects, indicating potential future overwriting.

^{This description was created by for cce9b0e. It will automatically update as commits are pushed.}

[ellipsis-dev]

👍 Looks good to me! Reviewed everything up to cce9b0e in 12 seconds

More details

• Looked at 20 lines of code in 1 files
• Skipped 0 files when reviewing.
• Skipped posting 1 drafted comments based on config settings.

1. skymp5-server/cpp/server_guest_lib/PartOne.cpp:314

• Draft comment:
  Consider storing lookupRes.rec->GetType() in a variable to avoid repeated calls. This applies to the newly added condition as well.
• Reason this comment was not posted:
  Confidence changes required: 50%
  The added condition checks for specific record types and logs a message if they are found. This is a reasonable addition, but it could be optimized by reducing repeated calls to lookupRes.rec->GetType().

Workflow ID: wflow_Df7yzqvouMHlqowm

---

You can customize Ellipsis with 👍 / 👎 feedback, review rules, user-specific overrides, quiet mode, and more.