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

Use pathbuffer reduce stack #14632

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

Commits on Nov 4, 2024

  1. Use lib_get_pathbuffer instead of stack variables

    Summary:
      Modified the usage logic, mainly introduced lib_get_pathbuffer and lib_put_pathbuffer
    
    Signed-off-by: chenrun1 <[email protected]>
    crafcat7 committed Nov 4, 2024
    Configuration menu
    Copy the full SHA
    6d41208 View commit details
    Browse the repository at this point in the history

Commits on Nov 5, 2024

  1. fix compile warning

    libs/libc/locale/lib_catalog.c:197:9: warning: implicit declaration of function ‘lib_get_pathbuffer’ [-Wimplicit-function-declaration]
      197 |   buf = lib_get_pathbuffer();
          |         ^~~~~~~~~~~~~~~~~~
    libs/libc/locale/lib_catalog.c:197:7: warning: assignment to ‘char *’ from ‘int’ makes pointer from integer without a cast [-Wint-conversion]
      197 |   buf = lib_get_pathbuffer();
          |       ^
    libs/libc/locale/lib_catalog.c:294:13: warning: implicit declaration of function ‘lib_put_pathbuffer’ [-Wimplicit-function-declaration]
      294 |             lib_put_pathbuffer(buf);
    
    Signed-off-by: zhangshoukui <[email protected]>
    Zhangshoukui authored and crafcat7 committed Nov 5, 2024
    Configuration menu
    Copy the full SHA
    b96d327 View commit details
    Browse the repository at this point in the history

Commits on Nov 6, 2024

  1. ffs/ffsl/ffsll:Repair in the kernel build cases multiple define problems

    Summary
     When CONFIG_HAVE_BUILTIN_XXX is enabled, use builtin functions to avoid implementation conflicts
    riscv-none-elf-ld: /github/workspace/sources/nuttx/staging/libc.a(exec_symtab.o): in function `ffs':
    /github/workspace/sources/nuttx/libs/libc/exec_symtab.c:81: multiple definition of `ffs'; /github/workspace/sources/nuttx/staging/libc.a(lib_ffs.o):/github/workspace/sources/nuttx/libs/libc/string/lib_ffs.c:51: first defined here
    make[1]: *** [Makefile:189: nuttx] Error 1
    make: *** [tools/Unix.mk:551: nuttx] Error 2
    
    Signed-off-by: chenrun1 <[email protected]>
    crafcat7 committed Nov 6, 2024
    Configuration menu
    Copy the full SHA
    3826dd0 View commit details
    Browse the repository at this point in the history