View Issue Details

IDProjectCategoryView StatusLast Update
0006475Dwarf FortressDwarf Mode -- Idle Behaviorpublic2014-07-12 19:56
ReporterQuietust Assigned Touser11 
PrioritynormalSeverityminorReproducibilityhave not tried
Status resolvedResolutionwon't fix 
Product Version0.34.11 
Summary0006475: Old acquaintances forgotten based on wrong criteria
DescriptionWhen a dwarf has not talked to an acquaintance for over a year, the acquaintance appears to be forgotten unless one of two conditions are true:

1. the dwarf itself has the unit flags "move_state", "dead", or "has_mood" set.
2. the acquaintance strength (number of talks) is greater than or equal to 30 (the threshold for "long-time acquaintance")

The first condition is almost definitely wrong - logically, it should be checking the flags on the acquaintance record itself, which would correspond to the flags for Friend, Grudge, and Bonded Animal (to prevent those from being forgotten).
Additional InformationThis bug appears to date back to version 0.28.181.40d, where it looks like it was trying to check the Friend and Grudge flags but looking in the wrong place.
Tagsbinary patch

Activities

Quietust

2014-04-29 18:25

reporter   ~0024739

Last edited: 2014-04-30 09:47

The following binary patches should fix this bug in version 0.34.11:

0.34.11 Win32 SDL:
0x47BA2B : F6 81 E0 00 00 00 07 -> F7 40 0C 07 00 00 00

0.34.11 Linux:
0x7F56AD : A9 8C -> AA 0C

0.34.11 OSX:
0x7D9C0C : 88 8C -> 8A 0C


The bug itself was discovered in the process of comparing the implementation of friendship between versions 0.23 and 0.34.

Quietust

2014-07-12 19:50

reporter   ~0026067

This may well no longer be an issue, since the old Friend system appears to have been scrapped entirely and replaced with something else that's stored at the hist_figure level.

user11

2014-07-12 19:56

  ~0026068

I'll go ahead an resolve it since it seems the system was rewritten. If you find that not to be the case, just PM a manager on the forums to reopen it.

Issue History

Date Modified Username Field Change
2014-04-29 14:57 Quietust New Issue
2014-04-29 18:25 Quietust Note Added: 0024739
2014-04-29 18:26 Quietust Tag Attached: binary patch
2014-04-29 18:33 Quietust Note Edited: 0024739
2014-04-30 00:15 user11 Assigned To => user11
2014-04-30 00:15 user11 Status new => acknowledged
2014-04-30 09:47 Quietust Note Edited: 0024739
2014-07-12 19:50 Quietust Note Added: 0026067
2014-07-12 19:56 user11 Note Added: 0026068
2014-07-12 19:56 user11 Status acknowledged => resolved
2014-07-12 19:56 user11 Resolution open => won't fix