[components][libc][posix][libdl] Optimization: Redundant calculations… #11062
Conversation
|
👋 感谢您对 RT-Thread 的贡献！Thank you for your contribution to RT-Thread! 为确保代码符合 RT-Thread 的编码规范，请在你的仓库中执行以下步骤运行代码格式化工作流（如果格式化CI运行失败）。 🛠 操作步骤 | Steps
完成后，提交将自动更新至 如有问题欢迎联系我们，再次感谢您的贡献！💐 |
📌 Code Review Assignment🏷️ Tag: componentsReviewers: Maihuanyi Changed Files (Click to expand)
🏷️ Tag: components_libcReviewers: GorrayLi mysterywolf Changed Files (Click to expand)
📊 Current Review Status (Last Updated: 2025-12-15 16:10 CST)
📝 Review Instructions
|
|
|
laidene
force-pushed
the
opt/components/libc/libdl
branch
from
b75d355 to
cbef04a
Compare
|
请作者签署下CLA，然后给出测试截图 |
测试记录在 qemu-vexpress-a9 中打开 CONFIG_RT_USING_MODULE，测试程序和截图。 创建一个动态库程序#ifndef __ADD_H__
#define __ADD_H__
#ifdef __cplusplus
extern "C" {
#endif
/**
* 加法函数
* @param a 第一个加数
* @param b 第二个加数
* @return 返回 a + b 的结果
*/
int add(int a, int b);
#ifdef __cplusplus
}
#endif
#endif /* __ADD_H__ */#include "add.h"
#include <rtthread.h>
/**
* 加法函数实现
* @param a 第一个加数
* @param b 第二个加数
* @return 返回 a + b 的结果
*/
int add(int a, int b)
{
rt_kprintf("add: %d + %d = %d\n", a, b, a + b);
return a + b;
}调用动态库测试程序/**
* 测试动态库加载和调用的示例程序
* 需要在RT-Thread中启用动态模块加载功能
*/
#include <rtthread.h>
#include <dlfcn.h>
#include <stdio.h>
#ifdef RT_USING_MODULE
/* 动态库路径 */
#define LIBADD_PATH "/lib/libadd.so"
/* 函数指针类型定义 */
typedef int (*add_func_t)(int a, int b);
static void test_add_lib(void)
{
void *handle;
add_func_t add_func;
int result;
const char *error;
rt_kprintf("Testing dynamic library: %s\n", LIBADD_PATH);
/* 打开动态库 */
handle = dlopen(LIBADD_PATH, RTLD_LAZY);
if (!handle) {
rt_kprintf("Error: Cannot open library %s: %s\n", LIBADD_PATH, dlerror());
return;
}
rt_kprintf("Library opened successfully\n");
/* 获取add函数地址 */
add_func = (add_func_t)dlsym(handle, "add");
rt_kprintf("Symbol 'add' found successfully\n");
/* 调用add函数 */
result = add_func(10, 20);
rt_kprintf("[c] add(10, 20) = %d\n", result);
result = add_func(100, 200);
rt_kprintf("[c] add(100, 200) = %d\n", result);
result = add_func(-5, 15);
rt_kprintf("[c] add(-5, 15) = %d\n", result);
/* 关闭动态库 */
dlclose(handle);
rt_kprintf("Library closed\n");
}
MSH_CMD_EXPORT(test_add_lib, Test dynamic library add function);
#else
#warning "RT_USING_MODULE is not defined, please enable dynamic module loading in Kconfig"
#endif /* RT_USING_MODULE */
运行截图
|
我已经签署并添加测试截图评论 |
3 participants
… in a for loop
拉取/合并请求描述：(PR description)
[
为什么提交这份PR (why to submit this PR)
避免在循环内重复计算 shstrab。Redundant calculations in a for loop.
你的解决方案是什么 (what is your solution)
移到循环外部，只计算一次。Move outside the loop to calculate only once.
请提供验证的bsp和config (provide the config and bsp)
请提供验证的bsp和config (provide the config and bsp)
BSP: bsp/qemu-vexpress-a9
.config: 无
action: 无
]
当前拉取/合并请求的状态 Intent for your PR
必须选择一项 Choose one (Mandatory):
代码质量 Code Quality：
我在这个拉取/合并请求中已经考虑了 As part of this pull request, I've considered the following:
#if 0代码，不包含已经被注释了的代码 All redundant code is removed and cleaned up