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

Build fixes #67

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Build fixes #67

wants to merge 2 commits into from

Conversation

illwieckz
Copy link
Contributor

tilp,gfm: fix build without NLS

gfm: fix truncation error

tilp_and_gfm/gfm/trunk/src/labels.c: In function ‘format_path’:
tilp_and_gfm/gfm/trunk/src/labels.c:91:37: error: ‘%6i’ directive output may be truncated writing between 6 and 10 bytes into a region of size 6 [-Werror=format-truncation=]
   91 |                 snprintf(str, 8, " (%6i)", n);
      |                                     ^~~

tilp_and_gfm/gfm/trunk/src/labels.c: In function ‘format_path’:
tilp_and_gfm/gfm/trunk/src/labels.c:91:37: error: ‘%6i’ directive output may be truncated writing between 6 and 10 bytes into a region of size 6 [-Werror=format-truncation=]
   91 |                 snprintf(str, 8, " (%6i)", n);
      |                                     ^~~
@adriweb
Copy link
Contributor

adriweb commented Aug 17, 2023

Regarding f04608f, LGTM, considering that's what I already did on the tilibs.

in fact, I had already committed this: b0a17e3 and improved it further here: f7baa39#diff-c37ff7dcdb829b8c28190b2453e8cc83bb9eace89dd8617c51afbfaf394f0a00R62

@illwieckz illwieckz changed the title Buildfixes Build fixes Aug 17, 2023
@illwieckz
Copy link
Contributor Author

What is the branch you prefer I develop against?

@adriweb
Copy link
Contributor

adriweb commented Aug 17, 2023

I'll let @debrouxl answer that, but master is probably fine.
Eventually the gtk3 branch should be merged though... 😅

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants