Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

libs: remove CatString2/CatString3 #336

Merged
merged 1 commit into from
Dec 12, 2020
Merged

libs: remove CatString2/CatString3 #336

merged 1 commit into from
Dec 12, 2020

Commits on Dec 10, 2020

  1. libs: remove CatString2/CatString3

    CatString2 and CatString3 provided a means of concatenating either two
    or three variables together.  However, the callers of these don't get to
    specify a format string, as in:
    
        "%s %s"
    
    which breaks compiling fvwm3 with:
    
        CFLAGS="-Wformat -Werror=format-security"
    
    Given these wrappers are inflexible, and that xasprintf() already exists
    in the codebase, switch to using that.
    
    Fixes #334
    ThomasAdam committed Dec 10, 2020
    Configuration menu
    Copy the full SHA
    805ec9f View commit details
    Browse the repository at this point in the history