Changeset 893

Show
Ignore:
Timestamp:
05/11/08 22:11:11 (2 months ago)
Author:
takkaria
Message:

Replace some "generic" parse errors with more specific ones.

Files:

Legend:

Unmodified
Added
Removed
Modified
Copied
Moved

  • trunk/src/init.h

    r892 r893  
    2828        PARSE_ERROR_MISSING_RECORD_HEADER, 
    2929        PARSE_ERROR_NON_SEQUENTIAL_RECORDS, 
     30        PARSE_ERROR_NOT_NUMBER, 
    3031        PARSE_ERROR_OBSOLETE_FILE, 
    3132        PARSE_ERROR_OUT_OF_BOUNDS, 
    …… 
    3435        PARSE_ERROR_TOO_MANY_ENTRIES, 
    3536        PARSE_ERROR_UNDEFINED_DIRECTIVE, 
     37        PARSE_ERROR_UNRECOGNISED_BLOW, 
    3638        PARSE_ERROR_UNRECOGNISED_TVAL, 
    3739        PARSE_ERROR_UNRECOGNISED_SVAL, 

  • trunk/src/init1.c

    r892 r893  
    568568 
    569569                /* Verify correct "colon" format */ 
    570                 if (buf[1] != ':') return (PARSE_ERROR_GENERIC); 
     570                if (buf[1] != ':') return (PARSE_ERROR_MISSING_COLON); 
    571571 
    572572 
    …… 
    818818 
    819819                /* Scan for the value */ 
    820                 if (1 != sscanf(buf+4, "%d", &max)) return (PARSE_ERROR_GENERIC); 
     820                if (1 != sscanf(buf+4, "%d", &max)) return (PARSE_ERROR_NOT_NUMBER); 
    821821 
    822822                /* Save the value */ 
    …… 
    830830 
    831831                /* Scan for the value */ 
    832                 if (1 != sscanf(buf+4, "%d", &max)) return (PARSE_ERROR_GENERIC); 
     832                if (1 != sscanf(buf+4, "%d", &max)) return (PARSE_ERROR_NOT_NUMBER); 
    833833 
    834834                /* Save the value */ 
    …… 
    842842 
    843843                /* Scan for the value */ 
    844                 if (1 != sscanf(buf+4, "%d", &max)) return (PARSE_ERROR_GENERIC); 
     844                if (1 != sscanf(buf+4, "%d", &max)) return (PARSE_ERROR_NOT_NUMBER); 
    845845 
    846846                /* Save the value */ 
    …… 
    854854 
    855855                /* Scan for the value */ 
    856                 if (1 != sscanf(buf+4, "%d", &max)) return (PARSE_ERROR_GENERIC); 
     856                if (1 != sscanf(buf+4, "%d", &max)) return (PARSE_ERROR_NOT_NUMBER); 
    857857 
    858858                /* Save the value */ 
    …… 
    866866 
    867867                /* Scan for the value */ 
    868                 if (1 != sscanf(buf+4, "%d", &max)) return (PARSE_ERROR_GENERIC); 
     868                if (1 != sscanf(buf+4, "%d", &max)) return (PARSE_ERROR_NOT_NUMBER); 
    869869 
    870870                /* Save the value */ 
    …… 
    879879 
    880880                /* Scan for the value */ 
    881                 if (1 != sscanf(buf+4, "%d", &max)) return (PARSE_ERROR_GENERIC); 
     881                if (1 != sscanf(buf+4, "%d", &max)) return (PARSE_ERROR_NOT_NUMBER); 
    882882 
    883883                /* Save the value */ 
    …… 
    892892 
    893893                /* Scan for the value */ 
    894                 if (1 != sscanf(buf+4, "%d", &max)) return (PARSE_ERROR_GENERIC); 
     894                if (1 != sscanf(buf+4, "%d", &max)) return (PARSE_ERROR_NOT_NUMBER); 
    895895 
    896896                /* Save the value */ 
    …… 
    904904 
    905905                /* Scan for the value */ 
    906                 if (1 != sscanf(buf+4, "%d", &max)) return (PARSE_ERROR_GENERIC); 
     906                if (1 != sscanf(buf+4, "%d", &max)) return (PARSE_ERROR_NOT_NUMBER); 
    907907 
    908908                /* Save the value */ 
    …… 
    916916 
    917917                /* Scan for the value */ 
    918                 if (1 != sscanf(buf+4, "%d", &max)) return (PARSE_ERROR_GENERIC); 
     918                if (1 != sscanf(buf+4, "%d", &max)) return (PARSE_ERROR_NOT_NUMBER); 
    919919 
    920920                /* Save the value */ 
    …… 
    928928 
    929929                /* Scan for the value */ 
    930                 if (1 != sscanf(buf+4, "%d", &max)) return (PARSE_ERROR_GENERIC); 
     930                if (1 != sscanf(buf+4, "%d", &max)) return (PARSE_ERROR_NOT_NUMBER); 
    931931 
    932932                /* Save the value */ 
    …… 
    940940 
    941941                /* Scan for the value */ 
    942                 if (1 != sscanf(buf+4, "%d", &max)) return (PARSE_ERROR_GENERIC); 
     942                if (1 != sscanf(buf+4, "%d", &max)) return (PARSE_ERROR_NOT_NUMBER); 
    943943 
    944944                /* Save the value */ 
    …… 
    952952 
    953953                /* Scan for the value */ 
    954                 if (1 != sscanf(buf+4, "%d", &max)) return (PARSE_ERROR_GENERIC); 
     954                if (1 != sscanf(buf+4, "%d", &max)) return (PARSE_ERROR_NOT_NUMBER); 
    955955 
    956956                /* Save the value */ 
    …… 
    964964 
    965965                /* Scan for the value */ 
    966                 if (1 != sscanf(buf+4, "%d", &max)) return (PARSE_ERROR_GENERIC); 
     966                if (1 != sscanf(buf+4, "%d", &max)) return (PARSE_ERROR_NOT_NUMBER); 
    967967 
    968968                /* Save the value */ 
    …… 
    976976 
    977977                /* Scan for the value */ 
    978                 if (1 != sscanf(buf+4, "%d", &max)) return (PARSE_ERROR_GENERIC); 
     978                if (1 != sscanf(buf+4, "%d", &max)) return (PARSE_ERROR_NOT_NUMBER); 
    979979 
    980980                /* Save the value */ 
    …… 
    988988 
    989989                /* Scan for the value */ 
    990                 if (1 != sscanf(buf+4, "%ld", &max)) return (PARSE_ERROR_GENERIC); 
     990                if (1 != sscanf(buf+4, "%ld", &max)) return (PARSE_ERROR_NOT_NUMBER); 
    991991 
    992992                /* Save the value */ 
    …… 
    10001000 
    10011001                /* Scan for the value */ 
    1002                 if (1 != sscanf(buf+4, "%ld", &max)) return (PARSE_ERROR_GENERIC); 
     1002                if (1 != sscanf(buf+4, "%ld", &max)) return (PARSE_ERROR_NOT_NUMBER); 
    10031003 
    10041004                /* Save the value */ 
    …… 
    10361036 
    10371037                /* Verify that colon */ 
    1038                 if (!s) return (PARSE_ERROR_GENERIC); 
     1038                if (!s) return (PARSE_ERROR_MISSING_COLON); 
    10391039 
    10401040                /* Nuke the colon, advance to the name */ 
    …… 
    11351135 
    11361136                /* Verify that colon */ 
    1137                 if (!s) return (PARSE_ERROR_GENERIC); 
     1137                if (!s) return (PARSE_ERROR_MISSING_COLON); 
    11381138 
    11391139                /* Nuke the colon, advance to the name */ 
    …… 
    11761176                /* Scan for the values */ 
    11771177                if (1 != sscanf(buf+2, "%d", 
    1178                                     &mimic)) return (PARSE_ERROR_GENERIC); 
     1178                                    &mimic)) return (PARSE_ERROR_NOT_NUMBER); 
    11791179 
    11801180                /* Save the values */ 
    …… 
    12711271 
    12721272        /* Error */ 
    1273         return (PARSE_ERROR_GENERIC); 
     1273        return (PARSE_ERROR_INVALID_FLAG); 
    12741274} 
    12751275 
    …… 
    13181318 
    13191319                /* Verify that colon */ 
    1320                 if (!s) return (PARSE_ERROR_GENERIC); 
     1320                if (!s) return (PARSE_ERROR_MISSING_COLON); 
    13211321 
    13221322                /* Nuke the colon, advance to the name */ 
    …… 
    15331533 
    15341534                        /* Parse this entry */ 
    1535                         if (0 != grab_one_kind_flag(k_ptr, s)) return (PARSE_ERROR_INVALID_FLAG); 
     1535                        if (0 != grab_one_kind_flag(k_ptr, s)) 
     1536                                return (PARSE_ERROR_INVALID_FLAG); 
    15361537 
    15371538                        /* Start the next entry */ 
    …… 
    16251626 
    16261627        /* Error */ 
    1627         return (PARSE_ERROR_GENERIC); 
     1628        return (PARSE_ERROR_INVALID_FLAG); 
    16281629} 
    16291630 
    …… 
    16491650 
    16501651                /* Verify that colon */ 
    1651                 if (!s) return (PARSE_ERROR_GENERIC); 
     1652                if (!s) return (PARSE_ERROR_MISSING_COLON); 
    16521653 
    16531654                /* Nuke the colon, advance to the name */ 
    …… 
    17521753 
    17531754                        /* Parse this entry */ 
    1754                         if (0 != grab_one_artifact_flag(a_ptr, s)) return (PARSE_ERROR_INVALID_FLAG); 
     1755                        if (0 != grab_one_artifact_flag(a_ptr, s)) 
     1756                                return (PARSE_ERROR_INVALID_FLAG); 
    17551757 
    17561758                        /* Start the next entry */ 
    …… 
    17661768                /* Find the colon after the name */ 
    17671769                s = strchr(buf + 2, ':'); 
    1768                 if (!s) return (PARSE_ERROR_GENERIC); 
     1770                if (!s) return (PARSE_ERROR_MISSING_COLON); 
    17691771 
    17701772                /* Nuke the colon, advance to the name */ 
    …… 
    18541856 
    18551857        /* Error */ 
    1856         return (PARSE_ERROR_GENERIC); 
     1858        return (PARSE_ERROR_INVALID_FLAG); 
    18571859} 
    18581860 
    …… 
    18821884 
    18831885                /* Verify that colon */ 
    1884                 if (!s) return (PARSE_ERROR_GENERIC); 
     1886                if (!s) return (PARSE_ERROR_MISSING_COLON); 
    18851887 
    18861888                /* Nuke the colon, advance to the name */ 
    …… 
    20122014 
    20132015                        /* Parse this entry */ 
    2014                         if (0 != grab_one_ego_item_flag(e_ptr, s)) return (PARSE_ERROR_INVALID_FLAG); 
     2016                        if (0 != grab_one_ego_item_flag(e_ptr, s)) 
     2017                                return (PARSE_ERROR_INVALID_FLAG); 
    20152018 
    20162019                        /* Start the next entry */ 
    …… 
    20622065 
    20632066        /* Failure */ 
    2064         return (PARSE_ERROR_GENERIC); 
     2067        return (PARSE_ERROR_INVALID_FLAG); 
    20652068} 
    20662069 
    …… 
    20842087 
    20852088        /* Failure */ 
    2086         return (PARSE_ERROR_GENERIC); 
     2089        return (PARSE_ERROR_INVALID_FLAG); 
    20872090} 
    20882091 
    …… 
    21102113 
    21112114                /* Verify that colon */ 
    2112                 if (!s) return (PARSE_ERROR_GENERIC); 
     2115                if (!s) return (PARSE_ERROR_MISSING_COLON); 
    21132116 
    21142117                /* Nuke the colon, advance to the name */ 
    …… 
    22312234 
    22322235                /* Oops, no more slots */ 
    2233                 if (i == MONSTER_BLOW_MAX) return (PARSE_ERROR_GENERIC); 
     2236                if (i == MONSTER_BLOW_MAX) 
     2237                        return (PARSE_ERROR_TOO_MANY_ENTRIES); 
    22342238 
    22352239                /* Analyze the first field */ 
    …… 
    22462250 
    22472251                /* Invalid method */ 
    2248                 if (!r_info_blow_method[n1]) return (PARSE_ERROR_GENERIC); 
     2252                if (!r_info_blow_method[n1]) return (PARSE_ERROR_UNRECOGNISED_BLOW); 
    22492253 
    22502254                /* Analyze the second field */ 
    …… 
    22612265 
    22622266                /* Invalid effect */ 
    2263                 if (!r_info_blow_effect[n2]) return (PARSE_ERROR_GENERIC); 
     2267                if (!r_info_blow_effect[n2]) return (PARSE_ERROR_UNRECOGNISED_BLOW); 
    22642268 
    22652269                /* Analyze the third field */ 
    …… 
    22972301 
    22982302                        /* Parse this entry */ 
    2299                         if (0 != grab_one_basic_flag(r_ptr, s)) return (PARSE_ERROR_INVALID_FLAG); 
     2303                        if (0 != grab_one_basic_flag(r_ptr, s)) 
     2304                                return (PARSE_ERROR_INVALID_FLAG); 
    23002305 
    23012306                        /* Start the next entry */ 
    …… 
    23752380 
    23762381        /* Error */ 
    2377         return (PARSE_ERROR_GENERIC); 
     2382        return (PARSE_ERROR_INVALID_FLAG); 
    23782383} 
    23792384 
    …… 
    24242429 
    24252430                /* Verify that colon */ 
    2426                 if (!s) return (PARSE_ERROR_GENERIC); 
     2431                if (!s) return (PARSE_ERROR_MISSING_COLON); 
    24272432 
    24282433                /* Nuke the colon, advance to the name */ 
    …… 
    24302435 
    24312436                /* Paranoia -- require a name */ 
    2432                 if (!*s) return (PARSE_ERROR_GENERIC); 
     2437                if (!*s) return (PARSE_ERROR_MISSING_FIELD); 
    24332438 
    24342439                /* Get the index */ 
    …… 
    24702475 
    24712476                        /* Verify that colon */ 
    2472                         if (!s) return (PARSE_ERROR_GENERIC); 
     2477                        if (!s) return (PARSE_ERROR_MISSING_COLON); 
    24732478 
    24742479                        /* Nuke the colon, advance to the subindex */ 
    …… 
    24932498 
    24942499                /* Verify text */ 
    2495                 if (!buf[1] || !buf[2]) return (PARSE_ERROR_GENERIC); 
     2500                if (!buf[1] || !buf[2]) return (PARSE_ERROR_MISSING_FIELD); 
    24962501 
    24972502                /* Scan and save the values */ 
    …… 
    25902595 
    25912596                        /* Parse this entry */ 
    2592                         if (0 != grab_one_racial_flag(pr_ptr, s)) return (PARSE_ERROR_INVALID_FLAG); 
     2597                        if (0 != grab_one_racial_flag(pr_ptr, s)) 
     2598                                return (PARSE_ERROR_INVALID_FLAG); 
    25932599 
    25942600                        /* Start the next entry */ 
    …… 
    26462652 
    26472653        /* Error */ 
    2648         return (PARSE_ERROR_GENERIC); 
     2654        return (PARSE_ERROR_INVALID_FLAG); 
    26492655} 
    26502656 
    …… 
    26732679 
    26742680                /* Verify that colon */ 
    2675                 if (!s) return (PARSE_ERROR_GENERIC); 
     2681                if (!s) return (PARSE_ERROR_MISSING_COLON); 
    26762682 
    26772683                /* Nuke the colon, advance to the name */ 
    …… 
    26792685 
    26802686                /* Paranoia -- require a name */ 
    2681                 if (!*s) return (PARSE_ERROR_GENERIC); 
     2687                if (!*s) return (PARSE_ERROR_MISSING_FIELD); 
    26822688 
    26832689                /* Get the index */ 
    …… 
    27232729 
    27242730                        /* Verify that colon */ 
    2725                         if (!s) return (PARSE_ERROR_GENERIC); 
     2731                        if (!s) return (PARSE_ERROR_MISSING_COLON); 
    27262732 
    27272733                        /* Nuke the colon, advance to the subindex */ 
    …… 
    27462752 
    27472753                /* Verify text */ 
    2748                 if (!buf[1] || !buf[2]) return (PARSE_ERROR_GENERIC); 
     2754                if (!buf[1] || !buf[2]) return (PARSE_ERROR_MISSING_FIELD); 
    27492755 
    27502756                /* Scan and save the values */ 
    …… 
    27592765 
    27602766                /* Verify text */ 
    2761                 if (!buf[1] || !buf[2]) return (PARSE_ERROR_GENERIC); 
     2767                if (!buf[1] || !buf[2]) return (PARSE_ERROR_MISSING_FIELD); 
    27622768 
    27632769                /* Scan and save the values */ 
    …… 
    30313037        { 
    30323038                /* Confirm the colon */ 
    3033                 if (buf[1] != ':') return PARSE_ERROR_GENERIC; 
     3039                if (buf[1] != ':') return PARSE_ERROR_MISSING_COLON; 
    30343040 
    30353041                /* Get the index */ 
    …… 
    31403146 
    31413147                /* Paranoia */ 
    3142                 if (!buf[2]) return (PARSE_ERROR_GENERIC); 
     3148                if (!buf[2]) return (PARSE_ERROR_MISSING_FIELD); 
    31433149                if (!buf[3]) return (PARSE_ERROR_GENERIC); 
    31443150                if (!buf[4]) return (PARSE_ERROR_GENERIC); 
    …… 
    31773183 
    31783184                /* Paranoia */ 
    3179                 if (!buf[1]) return (PARSE_ERROR_GENERIC); 
    3180                 if (!buf[2]) return (PARSE_ERROR_GENERIC); 
     3185                if (!buf[1]) return (PARSE_ERROR_MISSING_FIELD); 
     3186                if (!buf[2]) return (PARSE_ERROR_MISSING_FIELD); 
    31813187 
    31823188                /* Store the text */ 
    …… 
    32163222 
    32173223                /* Verify that colon */ 
    3218                 if (!s) return (PARSE_ERROR_GENERIC); 
     3224                if (!s) return (PARSE_ERROR_MISSING_COLON); 
    32193225 
    32203226                /* Nuke the colon, advance to the name */ 

  • trunk/src/init2.c

    r892 r893  
    239239        "missing record header", 
    240240        "non-sequential records", 
     241        "value not a number", 
    241242        "obsolete file", 
    242243        "value out of bounds", 
    …… 
    245246        "too many entries", 
    246247        "undefined directive", 
     248        "unrecognised blow", 
    247249        "unrecognised tval name", 
    248250        "unrecognised sval name",