1. 02 May, 2019 3 commits
    • Antonio Borneo's avatar
      log: remove "%m" from format strings by using strerror(errno) · 294ed97e
      Antonio Borneo authored
      
      
      The printf() format specifier "%m" is a glibc extension to print
      the string returned by strerror(errno). While supported by other
      libraries (e.g. uClibc and musl), it is not widely portable.
      
      In Wayland code the format string is often passed to a logging
      function that calls other syscalls before the conversion of "%m"
      takes place. If one of such syscall modifies the value in errno,
      the conversion of "%m" will incorrectly report the error string
      corresponding to the new value of errno.
      
      Remove all the occurrences of the specifier "%m" in Wayland code
      by using directly the string returned by strerror(errno).
      Signed-off-by: Antonio Borneo's avatarAntonio Borneo <borneo.antonio@gmail.com>
      294ed97e
    • Manuel Stoeckl's avatar
      tests: Verify that wayland_scanner can catch bad identifiers · 6db761d1
      Manuel Stoeckl authored and Simon Ser's avatar Simon Ser committed
      
      
      The test runs wayland_scanner on a set of XML protocol files which
      have malformed element names, and confirms that an error is produced
      and indicates the correct line.
      
      Copyright notifications are not included in the test files, as
      they are not code; of course, the project license still applies.
      Signed-off-by: default avatarManuel Stoeckl <code@mstoeckl.com>
      6db761d1
    • Manuel Stoeckl's avatar
      scanner: error when element names will not compile · 6ce4bbb3
      Manuel Stoeckl authored and Simon Ser's avatar Simon Ser committed
      
      
      This change checks that the "name" fields of the various structures in
      a Wayland protocol XML file will be converted into C identifiers that
      can be successfully compiled.
      
      For names which will be inserted as the prefix of an identifier
      enforce a match with [_a-zA-Z][_0-9a-zA-Z]* . For types only inserted
      as the suffix of an identifier (enum, entry), enforce a format of
      [_0-9a-zA-Z]+ .
      
      Unicode characters (and escape sequences like \u0394) are not allowed,
      because most older and some newer C compilers do not support them by
      default.
      
      For sake of simplicity, this patch does not check for collisions
      with reserved words or standard library names.
      Signed-off-by: default avatarManuel Stoeckl <code@mstoeckl.com>
      6ce4bbb3
  2. 27 Apr, 2019 1 commit
  3. 19 Apr, 2019 2 commits
  4. 12 Apr, 2019 1 commit
  5. 11 Apr, 2019 2 commits
  6. 08 Apr, 2019 1 commit
  7. 06 Apr, 2019 1 commit
  8. 21 Mar, 2019 1 commit
  9. 14 Mar, 2019 1 commit
  10. 13 Mar, 2019 3 commits
  11. 07 Mar, 2019 2 commits
    • Pekka Paalanen's avatar
      contributing: use Gitlab merge request workflow · 808bca89
      Pekka Paalanen authored
      The experience from Weston shows that the Gitlab merge request based workflow
      works really well. Recently there have also been issues with the mailing list
      that have made the email based workflow more painful than it used to be. Those
      issues might have been temporary or occasional, but they probably are only
      going to increase.
      
      The MR workflow is different, it has its issues
      (freedesktop/freedesktop#74) and we
      likely lose the explicit Reviewed-by etc. tags from commit messages, but it is
      also much easier to work with: no more whitespace damaged patches, lost email,
      setting up git-send-email; we gain automated CI before any human reviewer even
      looks at anything, and people can jump in to an ongoing discussion even if they
      weren't subscribed before.
      
      If you still want email, you can subscribe to that selectively(!) in Gitlab
      yourself.
      
      This text has been copied from Weston's CONTRIBUTING.md of the 5.0.91 release
      and slight...
      808bca89
    • Chris Billington's avatar
      wayland-util.h: add forward declaration for wl_object · 15cba8b0
      Chris Billington authored and Pekka Paalanen's avatar Pekka Paalanen committed
      The definition of wl_argument in wayland-util.h references wl_object,
      so wl_object ought to be defined in wayland-util.h. This resolves
      gitlab issue #78.
      
      Fixes: wayland/wayland#78
      
      Signed-off-by: Pekka Paalanen's avatarPekka Paalanen <pekka.paalanen@collabora.com>
      15cba8b0
  12. 05 Mar, 2019 1 commit
  13. 25 Feb, 2019 1 commit
    • Leonid Bobrov via wayland-devel's avatar
      tests: fix main symbol duplication · c70fd8a8
      Leonid Bobrov via wayland-devel authored and Pekka Paalanen's avatar Pekka Paalanen committed
      So far I got these errors before patching:
      
      libtool: link: cc -o .libs/headers-test -pthread -Wall -Wextra -Wno-unused-parameter -g -Wstrict-prototypes -Wmissing-prototypes -fvisibility=hidden -O2 -pipe tests/headers-test.o tests/headers-protocol-test.o tests/headers-protocol-core-test.o /tmp/obj/wayland-1.16.0/build-amd64/.libs/libtest-runner.a -L.libs -lwayland-client -lffi -lm -lwayland-server -lkvm -Wl,-rpath-link,/usr/local/lib
      ld: error: duplicate symbol: main
      >>> defined at headers-test.c:53 (/tmp/obj/wayland-1.16.0/wayland-1.16.0/tests/headers-test.c:53)
      >>>            tests/headers-test.o:(main)
      >>> defined at test-runner.c:377 (/tmp/obj/wayland-1.16.0/wayland-1.16.0/tests/test-runner.c:377)
      >>>            test-runner.o:(.text+0x250) in archive /tmp/obj/wayland-1.16.0/build-amd64/.libs/libtest-runner.a
      
      libtool: link: cc -o .libs/exec-fd-leak-checker -pthread -Wall -Wextra -Wno-unused-parameter -g -Wstrict-prototypes -Wmissing-prototypes -fvisibility=hidden -O2 ...
      c70fd8a8
  14. 20 Feb, 2019 1 commit
  15. 19 Feb, 2019 1 commit
  16. 05 Feb, 2019 1 commit
  17. 30 Jan, 2019 1 commit
  18. 29 Jan, 2019 2 commits
  19. 30 Nov, 2018 2 commits
  20. 29 Aug, 2018 6 commits
  21. 24 Aug, 2018 3 commits
  22. 17 Aug, 2018 3 commits