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

2020-08-04 10:06:13 +08:00
 sockpuppet9527

前情提要:/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. 为什么不用返回值啊?

手动狗头保命。纯吐槽

1186 次点击
所在节点    问与答
2 条回复
BingoXuan
2020-08-04 10:09:16 +08:00
可能作者不是一个人
cjw1115
2020-08-04 15:29:46 +08:00
哈哈 波兰人 code review 可是很严谨的

这是一个专为移动设备优化的页面(即为了让你能够在 Google 搜索结果里秒开这个页面),如果你希望参与 V2EX 社区的讨论,你可以继续到 V2EX 上打开本讨论主题的完整版本。

https://www.v2ex.com/t/695427

V2EX 是创意工作者们的社区,是一个分享自己正在做的有趣事物、交流想法,可以遇见新朋友甚至新机会的地方。

V2EX is a community of developers, designers and creative people.

© 2021 V2EX