Skip to content

tools: mksyscall fix compilation warning#18856

Merged
xiaoxiang781216 merged 1 commit intoapache:masterfrom
shtirlic:tools-mksyscall
May 9, 2026
Merged

tools: mksyscall fix compilation warning#18856
xiaoxiang781216 merged 1 commit intoapache:masterfrom
shtirlic:tools-mksyscall

Conversation

@shtirlic
Copy link
Copy Markdown
Contributor

@shtirlic shtirlic commented May 8, 2026

Summary

mksyscall.c:145:19: warning: initialization discards ‘const’ qualifier from pointer target type [-Wdiscarded-qualifiers]
  145 |   char *pactual = strchr(arg, '|');
      |                   ^~~~~~

Change to use const char* on pointers

Impact

none

Testing

 make
Create version.h
CPP:  /home/shtirlic/nuttx-space/nuttx/../picocalc-nx/scripts/memmap_default.ld-> /home/shtirlic/nuttx-space/nuttx/../picocalc-nx/scripts/memmapLD: nuttx
riscv-none-elf-ld: warning: /home/shtirlic/nuttx-space/nuttx/nuttx has a LOAD segment with RWX permissions
Memory region         Used Size  Region Size  %age Used
           FLASH:      822308 B         4 MB     19.61%
             RAM:       93100 B       512 KB     17.76%
       SCRATCH_X:           0 B         4 KB      0.00%
       SCRATCH_Y:           0 B         4 KB      0.00%
Generating: nuttx.uf2
Done.

mksyscall.c:145:19: warning: initialization discards ‘const’ qualifier
from pointer target type [-Wdiscarded-qualifiers]
  145 |   char *pactual = strchr(arg, '|');
      |                   ^~~~~~
Change to use const char* on pointers

Signed-off-by: Serg Podtynnyi <serg@podtynnyi.com>
@shtirlic shtirlic requested a review from xiaoxiang781216 as a code owner May 8, 2026 07:40
@github-actions github-actions Bot added Area: Tooling Size: XS The size of the change in this PR is very small labels May 8, 2026
@xiaoxiang781216 xiaoxiang781216 merged commit 9c1b2e1 into apache:master May 9, 2026
34 of 41 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Area: Tooling Size: XS The size of the change in this PR is very small

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants