---
- branch: MAIN
  date: Mon Jul 20 14:50:41 UTC 2020
  files:
  - new: '1.238'
    old: '1.237'
    path: src/usr.bin/make/parse.c
    pathrev: src/usr.bin/make/parse.c@1.238
    type: modified
  - new: '1.278'
    old: '1.277'
    path: src/usr.bin/make/var.c
    pathrev: src/usr.bin/make/var.c@1.278
    type: modified
  - new: '1.20'
    old: '1.19'
    path: src/usr.bin/make/unit-tests/modmisc.exp
    pathrev: src/usr.bin/make/unit-tests/modmisc.exp@1.20
    type: modified
  id: 20200720T145041Z.89ccb24ff119e4f4c3106e7de9f9336320b7c1e8
  log: |
    make(1): make modifier handling simpler

    Implementing a modifier such as :S or :M should not be concerned with
    separating the words of the resulting string.  Ideally this should be
    done in the same way by all modifiers.

    Before, the :R (filename root) modifier added a separator even if the
    resulting filename root was an empty string.  The chances that this
    change in behavior breaks anything are epsilon.

    The :@ modifier, if it appeared after a :ts modifier, did not use the
    word separator from the :ts modifier (which all other modifiers do) but
    always added a space.  This behavior has been preserved for now.  It's an
    unnecessary inconsistency though.

    In contrast to Buffer, the newly added SepBuf uses size_t for memory
    sizes and also uses the conventional parameter order (mem, memsize)
    instead of the unusual (memsize, mem).
  module: src
  subject: 'CVS commit: src/usr.bin/make'
  unixtime: '1595256641'
  user: rillig