… | |
… | |
13 | } |
13 | } |
14 | |
14 | |
15 | sub query { |
15 | sub query { |
16 | my ($ns, $flags, $text) = @_; |
16 | my ($ns, $flags, $text) = @_; |
17 | |
17 | |
18 | my $current = $Coro::current; |
18 | $ns->query ($flags, $text, Coro::rouse_cb); |
19 | $ns->query ($flags, $text, sub { $current->ready; $current = $_[0] }); |
19 | Coro::rouse_wait |
20 | Coro::schedule while ref $current; |
|
|
21 | |
|
|
22 | $current |
|
|
23 | } |
20 | } |
24 | |
21 | |
25 | sub can_cleanup { |
22 | sub can_cleanup { |
26 | my ($pl, $mtime) = @_; |
23 | my ($pl, $mtime) = @_; |
27 | |
24 | |
… | |
… | |
230 | $ns->send_drawinfo ( |
227 | $ns->send_drawinfo ( |
231 | "That username is currently used in another login session. " |
228 | "That username is currently used in another login session. " |
232 | . "Chose another, or wait till the other session has ended.", |
229 | . "Chose another, or wait till the other session has ended.", |
233 | cf::NDI_RED |
230 | cf::NDI_RED |
234 | ); |
231 | ); |
235 | } elsif ($user =~ /^[a-zA-Z0-9][a-zA-Z0-9\-_]{2,17}\z/) { |
232 | } elsif ($user =~ /^[a-zA-Z0-9][a-zA-Z0-9\-_]{2,19}\z/) { |
236 | last; |
233 | last; |
237 | } else { |
234 | } else { |
238 | $ns->send_drawinfo ( |
235 | $ns->send_drawinfo ( |
239 | "Your username contains illegal characters " |
236 | "Your username contains illegal characters " |
240 | . "(only a-z, A-Z and 0-9 are allowed), " |
237 | . "(only a-z, A-Z and 0-9 are allowed), " |
241 | . "or is not between 3 and 18 characters in length.", |
238 | . "or is not between 3 and 20 characters in length.", |
242 | cf::NDI_RED |
239 | cf::NDI_RED |
243 | ); |
240 | ); |
244 | } |
241 | } |
245 | Coro::Timer::sleep 0.4; |
242 | Coro::Timer::sleep 0.4; |
246 | } |
243 | } |
… | |
… | |
433 | Coro::Timer::sleep 0.2; |
430 | Coro::Timer::sleep 0.2; |
434 | } |
431 | } |
435 | |
432 | |
436 | $ob->reply (undef, "Welcome to Deliantra!"); |
433 | $ob->reply (undef, "Welcome to Deliantra!"); |
437 | |
434 | |
|
|
435 | # XXX: Workaround for delayed client ext protocol handshake |
|
|
436 | $pl->esrv_new_player; |
|
|
437 | |
438 | delete $pl->{deny_save}; |
438 | delete $pl->{deny_save}; |
439 | |
439 | |
440 | last; |
440 | last; |
441 | } |
441 | } |
442 | }); |
442 | }); |
… | |
… | |
520 | unless safe_spot $pl; |
520 | unless safe_spot $pl; |
521 | } |
521 | } |
522 | }, |
522 | }, |
523 | ); |
523 | ); |
524 | |
524 | |
525 | ############################################################################# |
|
|
526 | |
525 | |
527 | our $SCHEDULE_INTERVAL = $cf::CFG{player_schedule_interval} || 10; # time the player scheduler sleeps between runs |
|
|
528 | our $SAVE_TIMEOUT = $cf::CFG{player_save_interval} || 20; # save players every n seconds |
|
|
529 | |
|
|
530 | our $SCHEDULER = cf::async_ext { |
|
|
531 | $Coro::current->{desc} = "player scheduler"; |
|
|
532 | |
|
|
533 | while () { |
|
|
534 | Coro::EV::timer_once $SCHEDULE_INTERVAL; |
|
|
535 | |
|
|
536 | # this weird form of iteration over values is used because |
|
|
537 | # the hash changes underneath us frequently, and for |
|
|
538 | # keeps a direct reference to the value without (in 5.8 perls) |
|
|
539 | # keeping a reference, so this is prone to crashes or worse. |
|
|
540 | my @players = keys %cf::PLAYER; |
|
|
541 | for (@players) { |
|
|
542 | my $pl = $cf::PLAYER{$_} |
|
|
543 | or next; |
|
|
544 | $pl->valid or next; |
|
|
545 | |
|
|
546 | eval { |
|
|
547 | if ($pl->{last_save} + $SAVE_TIMEOUT <= $cf::RUNTIME) { |
|
|
548 | $pl->save; |
|
|
549 | |
|
|
550 | unless ($pl->active || $pl->ns) { |
|
|
551 | # check refcounts, this is tricky and needs to be adjusted to fit server internals |
|
|
552 | my $ob = $pl->ob; |
|
|
553 | |
|
|
554 | my $pl_ref = $pl->refcnt_cnt; |
|
|
555 | my $ob_ref = $ob->refcnt_cnt; |
|
|
556 | |
|
|
557 | ## pl_ref == $pl + ob->contr + %cf::PLAYER |
|
|
558 | ## ob_ref == $ob + pl->observe + simply being an object |
|
|
559 | if ($pl_ref == 3 && $ob_ref == 3) { |
|
|
560 | warn "player-scheduler destroy ", $ob->name;#d# |
|
|
561 | |
|
|
562 | # remove from sight and get fresh "copies" |
|
|
563 | $pl = delete $cf::PLAYER{$ob->name}; |
|
|
564 | $ob = $pl->ob; |
|
|
565 | |
|
|
566 | $pl->destroy; # destroys $ob |
|
|
567 | } else { |
|
|
568 | my $a_ = $pl->refcnt;#d# |
|
|
569 | my $b_ = $ob->refcnt;#d# |
|
|
570 | |
|
|
571 | warn "player-scheduler refcnt ", $ob->name, " pl $pl_ref/3 ob $ob_ref/3 (C pl $a_/1 ob $b_/2)\n";#d# |
|
|
572 | } |
|
|
573 | } |
|
|
574 | } |
|
|
575 | }; |
|
|
576 | warn $@ if $@; |
|
|
577 | cf::cede_to_tick; |
|
|
578 | }; |
|
|
579 | } |
|
|
580 | }; |
|
|
581 | |
|
|
582 | $SCHEDULER->prio (1); |
|
|
583 | |
|
|