V2EX = way to explore
V2EX 是一个关于分享和探索的地方
现在注册
已注册用户请  登录
V2EX 提问指南
sockpuppet9527
V2EX  ›  问与答

今天看到 libfuse 的接口,我傻了 [手动狗头]

  •  
  •   sockpuppet9527 · 2020-08-04 10:06:13 +08:00 · 1133 次点击
    这是一个创建于 1370 天前的主题,其中的信息可能已经有所发展或是发生改变。

    前情提要:/t/693941

    
    int (*read) (const char *, char *, size_t, off_t,
    	struct fuse_file_info *);
    	
    int (*fallocate) (const char *, int, off_t, off_t,
    	struct fuse_file_info *);
                  
    ssize_t (*copy_file_range) (const char *path_in,
    	struct fuse_file_info *fi_in,
    	off_t offset_in, const char *path_out,
    	struct fuse_file_info *fi_out,
    	off_t offset_out, size_t size, int flags);
        
    

    都是在 fuse_operations 接口中的方法,我想如果让我波兰的同事来提 cr 肯定是一万个 commons

    1. 为什么 read 用 size_t 表示 size,在 fallocate 用 off_t 表示 size 啊?
    2. 为什么有些地方参数有名字,有些地方不加啊?
    3. 为什么注释格式不一样啊?
    4. 为什么不用返回值啊?

    手动狗头保命。纯吐槽

    2 条回复    2020-08-04 15:29:46 +08:00
    BingoXuan
        1
    BingoXuan  
       2020-08-04 10:09:16 +08:00
    可能作者不是一个人
    cjw1115
        2
    cjw1115  
       2020-08-04 15:29:46 +08:00
    哈哈 波兰人 code review 可是很严谨的
    关于   ·   帮助文档   ·   博客   ·   API   ·   FAQ   ·   我们的愿景   ·   实用小工具   ·   755 人在线   最高记录 6543   ·     Select Language
    创意工作者们的社区
    World is powered by solitude
    VERSION: 3.9.8.5 · 25ms · UTC 20:46 · PVG 04:46 · LAX 13:46 · JFK 16:46
    Developed with CodeLauncher
    ♥ Do have faith in what you're doing.