empire/empserver
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Fix read not to split production report when update is slow

Browse source 
Adjacent telegrams are squashed together if type and sender are the
same, and the timestamp is "close enough".  This is done in two
places: rea() and typed_wu().  They're inconsistent: typed_wu()
ignores the timestamp for production reports since Empire 2, but rea()
doesn't.

Record typed_wu()'s decision in new telstr member tel_cont.  Use it in
rea().
This commit is contained in:
Markus Armbruster 2011-11-21 07:26:02 +01:00
parent a0b82b8d46
commit 17223e8fe2
3 changed files with 8 additions and 23 deletions
Download patch file Download diff file Expand all files Collapse all files

4
src/lib/subs/wu.c
View file

@ -28,6 +28,7 @@
*
* Known contributors to this file:
* Steve McClure, 2000
* Markus Armbruster, 2005-2011
*/
#include <config.h>
@ -145,6 +146,7 @@ typed_wu(natid from, natid to, char *message, int type)
len = strlen(message);
tel.tel_length = len;
tel.tel_type = type;
tel.tel_cont = !telegram_is_new(to, &tel);
iov[0].iov_base = &tel;
iov[0].iov_len = sizeof(tel);
iov[1].iov_base = message;
@ -169,7 +171,7 @@ typed_wu(natid from, natid to, char *message, int type)
putnat(np);
}
} else {
if (!np->nat_tgms || telegram_is_new(to, &tel)) {
if (!np->nat_tgms || !tel.tel_cont) {
np->nat_tgms++;
putnat(np);
if (np->nat_flags & NF_INFORM) {