[Freeciv-Dev] (PR#6380) remove is_real and is_valid sanity variables

["John Wheeler" <jdwheeler42@xxxxxxxxx>]

[jdorje - Thu Oct  2 23:47:02 2003]:

> Most places in the code use this form:
> 
>    if (!normalize_map_pos(&x, &y)) {
>      assert(0);
>    }

Hmm... for a quick solution, would "assert(normalize_map_pos(&x, &y);" 
work?  I hate getting 'assert(0)' as an error message, because it 
gives absolutely no indication what the problem is.  What I'd much 
rather see is something like:

    if (!normalize_map_pos(&x, &y)) {
      die("Map position %d,%d not normalized.", x, y);
    }

-- 

++JohnWheeler