Menu

#485 sdasgb Fix N flag help to current behavior

None
closed-accepted
None
5
2025-01-18
2025-01-18
bbbbbr
No

In commit 15124 which merged some changes for ASxxxx v5.50 the -n flag was removed in order to resolve a clash with an upstream flag of the same letter. The previous -n flag behavior is still accessible via upper case -N, however the help text and function comments were not updated to reflect this change.

This is the relevant comment which turns off the previous -n flag behavior.

/trunk/sdcc/sdas/asxxsrc/asmain.c
@469
//                                case 'n':
                                /* TODO: collides with new Specify number of assembler scanning passes */
                                case 'N':
                                        nflag = 1;
                                        break;

These are the two areas which need updating to indicate -N instead of -n

@239

 *              int     nflag           -n, don't resolve global assigned value symbols flag

@2536
        "  -n   Don't resolve global assigned value symbols",

The attached patch makes those changes.

1 Attachments

Discussion

  • Benedikt Freisen

    • status: open --> closed-accepted
    • assigned_to: Benedikt Freisen
    • Group: -->
     
  • Benedikt Freisen

    Applied to the "next" branch in [r15231]. Thanks.

     

    Related

    Commit: [r15231]


Log in to post a comment.

MongoDB Logo MongoDB