Alexander_KS/SAPFOR
4
2
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Ошибка при создании массивов #73

Closed
xnpster wants to merge 1 commits from fix_array_analysis into master
pull from: fix_array_analysis
merge into: Alexander_KS:master
Conversation 2 Commits 1 Files Changed 1 +1 -1
xnpster commented 2026-01-31 17:58:49 +00:00
Collaborator
Copy Link

Не знаю, как правильно фиксить этот баг. Можно пофиксить так, и надеяться, что третий элемент этого ключа нигде не используется. Можно подойти по-другому и вообще переписать этот механизм, завести отдельную структуру под хранение этого маппинга. Вообще странная семантика у этих ключей, на первой позиции находится то место в коде, то порядковый номер в common блоке. Если присмотреться, то можно спокойно написать контрпример с локальным массивом common_*, который сломает эту систему.

Не знаю, как правильно фиксить этот баг. Можно пофиксить так, и надеяться, что третий элемент этого ключа нигде не используется. Можно подойти по-другому и вообще переписать этот механизм, завести отдельную структуру под хранение этого маппинга. Вообще странная семантика у этих ключей, на первой позиции находится то место в коде, то порядковый номер в common блоке. Если присмотреться, то можно спокойно написать контрпример с локальным массивом common_*, который сломает эту систему.
xnpster added 1 commit 2026-01-31 17:58:49 +00:00
fix getUniqName 998fbc3036
Alexander_KS commented 2026-01-31 18:01:01 +00:00
Owner
Copy Link

А какая проблема фиксится? которая обсуждалась - когда массивы из комон блоков накладываются? планировал завтра это посмореть

А какая проблема фиксится? которая обсуждалась - когда массивы из комон блоков накладываются? планировал завтра это посмореть
Alexander_KS commented 2026-02-01 09:28:44 +00:00
Owner
Copy Link

исправил по нормальному, возможно, будут еще исправления. Если обнаружатся ошибки, пиши в баг трекер с тестовым кодом.

исправил по нормальному, возможно, будут еще исправления. Если обнаружатся ошибки, пиши в баг трекер с тестовым кодом.
Alexander_KS closed this pull request 2026-02-01 09:28:44 +00:00

Pull request closed

Please reopen this pull request to perform a merge.
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
No items
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
abrosim0v Dudarenko EgorMayorov gr_gusev IgorBobrov oleg_nikitin xnpster Alexander_KS
No Assignees
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: Alexander_KS/SAPFOR#73
Delete Branch "fix_array_analysis"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?