ViewVC Help
View File | Revision Log | Show Annotations | Download File
/cvs/deliantra/server/lib/cf.pm
(Generate patch)

Comparing deliantra/server/lib/cf.pm (file contents):
Revision 1.119 by root, Tue Jan 2 08:26:42 2007 UTC vs.
Revision 1.120 by root, Tue Jan 2 11:08:36 2007 UTC

… …
179=cut 179=cut
180 180
181sub to_json($) { 181sub to_json($) {
182 $JSON::Syck::ImplicitUnicode = 0; # work around JSON::Syck bugs 182 $JSON::Syck::ImplicitUnicode = 0; # work around JSON::Syck bugs
183 JSON::Syck::Dump $_[0] 183 JSON::Syck::Dump $_[0]
184}
185
186=item my $guard = cf::guard { BLOCK }
187
188Run the given callback when the guard object gets destroyed (useful for
189coroutine cancellations).
190
191You can call C<< ->cancel >> on the guard object to stop the block from
192being executed.
193
194=cut
195
196sub guard(&) {
197 bless \(my $cb = $_[0]), cf::guard::;
198}
199
200sub cf::guard::cancel {
201 ${$_[0]} = sub { };
202}
203
204sub cf::guard::DESTROY {
205 ${$_[0]}->();
206}
207
208=item cf::lock_wait $string
209
210Wait until the given lock is available. See cf::lock_acquire.
211
212=item my $lock = cf::lock_acquire $string
213
214Wait until the given lock is available and then acquires it and returns
215a guard object. If the guard object gets destroyed (goes out of scope,
216for example when the coroutine gets canceled), the lock is automatically
217returned.
218
219Lock names should begin with a unique identifier (for example, find_map
220uses map_find and load_map uses map_load).
221
222=cut
223
224our %LOCK;
225
226sub lock_wait($) {
227 my ($key) = @_;
228
229 # wait for lock, if any
230 while ($LOCK{$key}) {
231 push @{ $LOCK{$key} }, $Coro::current;
232 Coro::schedule;
233 }
234}
235
236sub lock_acquire($) {
237 my ($key) = @_;
238
239 # wait, to be sure we are not locked
240 lock_wait $key;
241
242 $LOCK{$key} = [];
243
244 cf::guard {
245 # wake up all waiters, to be on the safe side
246 $_->ready for @{ delete $LOCK{$key} };
247 }
184} 248}
185 249
186=item cf::sync_job { BLOCK } 250=item cf::sync_job { BLOCK }
187 251
188The design of crossfire+ requires that the main coro ($Coro::main) is 252The design of crossfire+ requires that the main coro ($Coro::main) is
… …
1118 #warn "find_map<$path,$origin>\n";#d# 1182 #warn "find_map<$path,$origin>\n";#d#
1119 1183
1120 $path = new cf::path $path, $origin && $origin->path; 1184 $path = new cf::path $path, $origin && $origin->path;
1121 my $key = $path->as_string; 1185 my $key = $path->as_string;
1122 1186
1187 cf::lock_wait "map_find:$key";
1188
1123 $cf::MAP{$key} || do { 1189 $cf::MAP{$key} || do {
1190 my $guard = cf::lock_acquire "map_find:$key";
1191
1124 # do it the slow way 1192 # do it the slow way
1125 my $map = try_load_header $path->save_path; 1193 my $map = try_load_header $path->save_path;
1126 1194
1127 if ($map) { 1195 if ($map) {
1128 # safety 1196 # safety
… …
1161} 1229}
1162 1230
1163sub load { 1231sub load {
1164 my ($self) = @_; 1232 my ($self) = @_;
1165 1233
1234 my $path = $self->{path};
1235 my $guard = cf::lock_acquire "map_load:" . $path->as_string;
1236
1166 return if $self->in_memory != cf::MAP_SWAPPED; 1237 return if $self->in_memory != cf::MAP_SWAPPED;
1167 1238
1168 $self->in_memory (cf::MAP_LOADING); 1239 $self->in_memory (cf::MAP_LOADING);
1169
1170 my $path = $self->{path};
1171 1240
1172 $self->alloc; 1241 $self->alloc;
1173 $self->load_objects ($self->{load_path}, 1) 1242 $self->load_objects ($self->{load_path}, 1)
1174 or return; 1243 or return;
1175 1244
… …
1408=cut 1477=cut
1409 1478
1410sub cf::object::player::enter_link { 1479sub cf::object::player::enter_link {
1411 my ($self) = @_; 1480 my ($self) = @_;
1412 1481
1482 $self->deactivate_recursive;
1483
1413 return if $self->map == $LINK_MAP; 1484 return if $self->map == $LINK_MAP;
1414 1485
1415 $self->{_link_pos} = [$self->map->{path}, $self->x, $self->y] 1486 $self->{_link_pos} ||= [$self->map->{path}, $self->x, $self->y]
1416 if $self->map; 1487 if $self->map;
1417 1488
1418 $self->enter_map ($LINK_MAP, 20, 20); 1489 $self->enter_map ($LINK_MAP, 20, 20);
1419 $self->deactivate_recursive;
1420} 1490}
1421 1491
1422sub cf::object::player::leave_link { 1492sub cf::object::player::leave_link {
1423 my ($self, $map, $x, $y) = @_; 1493 my ($self, $map, $x, $y) = @_;
1424 1494
… …
1446 $map->load; 1516 $map->load;
1447 1517
1448 $self->activate_recursive; 1518 $self->activate_recursive;
1449 $self->enter_map ($map, $x, $y); 1519 $self->enter_map ($map, $x, $y);
1450} 1520}
1521
1522cf::player->attach (
1523 on_logout => sub {
1524 my ($pl) = @_;
1525
1526 # abort map switching before logout
1527 if ($pl->ob->{_link_pos}) {
1528 cf::sync_job {
1529 $pl->ob->leave_link
1530 };
1531 }
1532 },
1533 on_login => sub {
1534 my ($pl) = @_;
1535
1536 # try to abort aborted map switching on player login :)
1537 # should happen only on crashes
1538 if ($pl->ob->{_link_pos}) {
1539 $pl->ob->enter_link;
1540 Coro::async {
1541 # we need this sleep as the login has a concurrent enter_exit running
1542 # and this sleep increases chances of the player not ending up in scorn
1543 Coro::Timer::sleep 1;
1544 $pl->ob->leave_link;
1545 };
1546 }
1547 },
1548);
1451 1549
1452=item $player_object->goto_map ($path, $x, $y) 1550=item $player_object->goto_map ($path, $x, $y)
1453 1551
1454=cut 1552=cut
1455 1553

Diff Legend

– Removed lines
+ Added lines
< Changed lines
> Changed lines