Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added #include <stdbool.h> in avif.c #6380

Closed
wants to merge 1 commit into from

Conversation

Germano0
Copy link
Contributor

In C programming language you must use stdbool.h in order to be able to use boolean type.

/builddir/build/BUILD/darktable-3.2.1/src/imageio/format/avif.c: In function 'write_image':
/builddir/build/BUILD/darktable-3.2.1/src/imageio/format/avif.c:300:5: error: unknown type name 'bool'
  300 |     bool use_icc = false;
      |     ^~~~
/builddir/build/BUILD/darktable-3.2.1/src/imageio/format/avif.c:300:20: error: 'false' undeclared (first use in this function)
  300 |     bool use_icc = false;
      |                    ^~~~~
/builddir/build/BUILD/darktable-3.2.1/src/imageio/format/avif.c:300:20: note: each undeclared identifier is reported only once for each function it appears in
/builddir/build/BUILD/darktable-3.2.1/src/imageio/format/avif.c:353:17: error: 'true' undeclared (first use in this function)
  353 |       use_icc = true;
      |                 ^~~~
gmake[2]: *** [src/imageio/format/CMakeFiles/avif_format.dir/build.make:85: src/imageio/format/CMakeFiles/avif_format.dir/avif.c.o] Error 1
gmake[2]: Leaving directory '/builddir/build/BUILD/darktable-3.2.1/x86_64-redhat-linux-gnu/x86_64-redhat-linux-gnu'
gmake[1]: *** [CMakeFiles/Makefile2:5668: src/imageio/format/CMakeFiles/avif_format.dir/all] Error 2
gmake[1]: *** Waiting for unfinished jobs....

From https://kojipkgs.fedoraproject.org//work/tasks/7672/52257672/build.log

In C programming language you must use stdbool.h in order to be able to use boolean type.
```
/builddir/build/BUILD/darktable-3.2.1/src/imageio/format/avif.c: In function 'write_image':
/builddir/build/BUILD/darktable-3.2.1/src/imageio/format/avif.c:300:5: error: unknown type name 'bool'
  300 |     bool use_icc = false;
      |     ^~~~
/builddir/build/BUILD/darktable-3.2.1/src/imageio/format/avif.c:300:20: error: 'false' undeclared (first use in this function)
  300 |     bool use_icc = false;
      |                    ^~~~~
/builddir/build/BUILD/darktable-3.2.1/src/imageio/format/avif.c:300:20: note: each undeclared identifier is reported only once for each function it appears in
/builddir/build/BUILD/darktable-3.2.1/src/imageio/format/avif.c:353:17: error: 'true' undeclared (first use in this function)
  353 |       use_icc = true;
      |                 ^~~~
gmake[2]: *** [src/imageio/format/CMakeFiles/avif_format.dir/build.make:85: src/imageio/format/CMakeFiles/avif_format.dir/avif.c.o] Error 1
gmake[2]: Leaving directory '/builddir/build/BUILD/darktable-3.2.1/x86_64-redhat-linux-gnu/x86_64-redhat-linux-gnu'
gmake[1]: *** [CMakeFiles/Makefile2:5668: src/imageio/format/CMakeFiles/avif_format.dir/all] Error 2
gmake[1]: *** Waiting for unfinished jobs....
```
From https://kojipkgs.fedoraproject.org//work/tasks/7672/52257672/build.log
@TurboGit
Copy link
Member

This is already fixed in master, so please update your branch. And the proper fix was to use gboolean anyway.

@TurboGit TurboGit closed this Sep 26, 2020
@TurboGit TurboGit self-requested a review September 26, 2020 06:35
@TurboGit TurboGit added the bug: invalid the bug is not a bug, but a feature label Sep 26, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug: invalid the bug is not a bug, but a feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants