Redirect some headers noticed by the compiler #64

已合并
SlavekB 4 年前 将 1 次代码提交从 feat/header-redirections 合并至 master
selk 评论于 4 年前
协作者
/usr/include/sys/poll.h:1:2: warning: #warning redirecting incorrect #include <sys/poll.h> to <poll.h> [-Wcpp]
    1 | #warning redirecting incorrect #include <sys/poll.h> to <poll.h>
/usr/include/sys/signal.h:1:2: warning: #warning redirecting incorrect #include <sys/signal.h> to <signal.h> [-Wcpp]
    1 | #warning redirecting incorrect #include <sys/signal.h> to <signal.h>

Note: <sys/signal.h> has been removed from the tdeio/tdeio/connection.cpp source - it was already declared.
``` /usr/include/sys/poll.h:1:2: warning: #warning redirecting incorrect #include <sys/poll.h> to <poll.h> [-Wcpp] 1 | #warning redirecting incorrect #include <sys/poll.h> to <poll.h> /usr/include/sys/signal.h:1:2: warning: #warning redirecting incorrect #include <sys/signal.h> to <signal.h> [-Wcpp] 1 | #warning redirecting incorrect #include <sys/signal.h> to <signal.h> Note: <sys/signal.h> has been removed from the tdeio/tdeio/connection.cpp source - it was already declared. ```
SlavekB 评论于 4 年前
所有者

Here's an interesting situation with sys/poll.h × poll.h:

  • On Linux (glibc) poll.h contains #include <sys/poll.h>, so the real file is really in `sys/poll.h
  • On FreeBSD poll.h is symlink to sys/poll.h, so the real file is really in sys/poll.h.

The mentioned incorrect seems to be on many systems as a real true file location. That's why I'm a little hesitant about what's right.

In any case, the test on Debian and FreeBSD is without problems.

Here's an interesting situation with `sys/poll.h` × `poll.h`: * On Linux (glibc) `poll.h` contains `#include <sys/poll.h>`, so the real file is really in `sys/poll.h * On FreeBSD `poll.h` is symlink to `sys/poll.h`, so the real file is really in `sys/poll.h`. The mentioned *incorrect* seems to be on many systems as a real true file location. That's why I'm a little hesitant about what's right. In any case, the test on Debian and FreeBSD is without problems.
selk 评论于 4 年前
发布者
协作者
https://pubs.opengroup.org/onlinepubs/9699919799/basedefs/poll.h.html#tag_13_34
SlavekB 评论于 4 年前
所有者

Please can you rebase to the current HEAD so that your GPG signature can be preserved when merged?

Please can you rebase to the current HEAD so that your GPG signature can be preserved when merged?
selk 评论于 4 年前
发布者
协作者

Done. 👍

Done. :+1:
SlavekB4 年前 关闭此合并请求
SlavekB4 年前 删除了分支 feat/header-redirections
SlavekB4 年前 添加了里程碑 R14.0.8 release
该合并请求已作为 c2adce2e13 被合并。
登录 并参与到对话中。
无审核者
未选择里程碑
未指派成员
2 名参与者
通知
到期时间

未设置到期时间。

依赖工单

没有设置依赖项。

参考:TDE/tdelibs#64
正在加载...
这个人很懒,什么都没留下。