… | |
… | |
55 | } |
55 | } |
56 | |
56 | |
57 | sub check_clean_save { |
57 | sub check_clean_save { |
58 | my ($pl) = @_; |
58 | my ($pl) = @_; |
59 | |
59 | |
60 | unless (delete $pl->{clean_save}) { |
60 | if (my $time = delete $pl->{unclean_save}) { |
|
|
61 | $pl->ns->send_drawinfo ( |
|
|
62 | "You didn't use a savebed to leave this realm. This is very dangerous, " |
|
|
63 | . "as lots of things could happen when you leave by other means, such as cave-ins, " |
|
|
64 | . "or monsters suddenly snapping your body. Better use a savebed next time.", |
|
|
65 | cf::NDI_RED |
|
|
66 | ); |
61 | #d#TODO |
67 | #d#TODO |
62 | } |
68 | } |
63 | } |
69 | } |
64 | |
70 | |
65 | # delete a player directory, be non-blocking AND synchronous... |
71 | # delete a player directory, be non-blocking AND synchronous... |
… | |
… | |
156 | nuke_str $pass; |
162 | nuke_str $pass; |
157 | # password matches, wonderful |
163 | # password matches, wonderful |
158 | my $pl = cf::player::find $user or next; |
164 | my $pl = cf::player::find $user or next; |
159 | $pl->connect ($ns); |
165 | $pl->connect ($ns); |
160 | check_clean_save $pl; |
166 | check_clean_save $pl; |
161 | $pl->{clean_save} = 1; |
|
|
162 | last; |
167 | last; |
163 | } elsif (can_cleanup $pl, $mtime) { |
168 | } elsif (can_cleanup $pl, $mtime) { |
164 | Coro::Timer::sleep 1; |
169 | Coro::Timer::sleep 1; |
165 | |
170 | |
166 | $ns->send_drawinfo ( |
171 | $ns->send_drawinfo ( |
… | |
… | |
328 | |
333 | |
329 | if ($cleanly) { |
334 | if ($cleanly) { |
330 | $_->ob->message ("$name left the game.", cf::NDI_DK_ORANGE | cf::NDI_UNIQUE) for cf::player::list; |
335 | $_->ob->message ("$name left the game.", cf::NDI_DK_ORANGE | cf::NDI_UNIQUE) for cf::player::list; |
331 | } else { |
336 | } else { |
332 | $_->ob->message ("$name uncerimoniously disconnected.", cf::NDI_DK_ORANGE | cf::NDI_UNIQUE) for cf::player::list; |
337 | $_->ob->message ("$name uncerimoniously disconnected.", cf::NDI_DK_ORANGE | cf::NDI_UNIQUE) for cf::player::list; |
333 | delete $pl->{clean_save}; |
338 | $pl->{unclean_save} = $cf::RUNTIME; |
334 | } |
339 | } |
335 | }, |
340 | }, |
336 | ); |
341 | ); |
337 | |
342 | |
338 | cf::client->attach ( |
343 | cf::client->attach ( |
… | |
… | |
343 | |
348 | |
344 | our $SCHEDULE_INTERVAL = 10; # time the player scheduler sleeps between runs |
349 | our $SCHEDULE_INTERVAL = 10; # time the player scheduler sleeps between runs |
345 | our $SAVE_TIMEOUT = 20; # save players every n seconds |
350 | our $SAVE_TIMEOUT = 20; # save players every n seconds |
346 | |
351 | |
347 | our $SCHEDULER = cf::async_ext { |
352 | our $SCHEDULER = cf::async_ext { |
|
|
353 | my $schedule_interval = Coro::Event->timer (after => 1, interval => $SCHEDULE_INTERVAL); |
348 | while () { |
354 | while () { |
349 | Coro::Timer::sleep $SCHEDULE_INTERVAL; |
355 | $schedule_interval->next; |
350 | |
356 | |
351 | # this weird form of iteration over values is used because |
357 | # this weird form of iteration over values is used because |
352 | # the hash changes underneath us frequently, and for |
358 | # the hash changes underneath us frequently, and for |
353 | # keeps a direct reference to the value without (in 5.8 perls) |
359 | # keeps a direct reference to the value without (in 5.8 perls) |
354 | # keeping a reference, so this is prone to crashes or worse. |
360 | # keeping a reference, so this is prone to crashes or worse. |
… | |
… | |
373 | my $pl_ref = $pl->refcnt_cnt; |
379 | my $pl_ref = $pl->refcnt_cnt; |
374 | my $ob_ref = $ob->refcnt_cnt; |
380 | my $ob_ref = $ob->refcnt_cnt; |
375 | |
381 | |
376 | ## pl_ref == one from object + one from cf::PLAYER |
382 | ## pl_ref == one from object + one from cf::PLAYER |
377 | ## ob_ref == one from simply being an object |
383 | ## ob_ref == one from simply being an object |
|
|
384 | #TODO: the above should be correct, understand the 1/0, respectively |
378 | if ($pl_ref == 2 && $ob_ref == 1) { |
385 | if ($pl_ref == 1 && $ob_ref == 0) { |
379 | warn "player-scheduler destroy ", $ob->name;#d# |
386 | warn "player-scheduler destroy ", $ob->name;#d# |
380 | |
387 | |
381 | # remove from sight and get fresh "copies" |
388 | # remove from sight and get fresh "copies" |
382 | $pl = delete $cf::PLAYER{$ob->name}; |
389 | $pl = delete $cf::PLAYER{$ob->name}; |
383 | $ob = $pl->ob; |
390 | $ob = $pl->ob; |