vlc_fourcc_GetChromaDescription() has no side effects.

This could loose events and was theoretically undefined.

Else the if(end) were meaningless

Ternary: a ? b : b

This reverts commit d073d3be.

```--
access/dvdnav.c: In function 'ProbeDVD':
access/dvdnav.c:1469:6: error: implicit declaration of function 'read' [-Werror=implicit-function-declaration]
      || read( fd, iso_dsc, sizeof (iso_dsc) ) < (int)sizeof (iso_dsc)
      ^
access/dvdnav.c:1481:5: error: implicit declaration of function 'close' [-Werror=implicit-function-declaration]
     close( fd );
     ^
```

--
Signed-off-by: Rémi Denis-Courmont <remi@remlab.net>

and add a missing one

Close #12753

Thanks to Ruito

The same caveats apply as for Windows legacy (and ALSA) outputs:
- Change latency is high and noticeable.
- Change between less than 100% and more than 100% occurs in two steps.
- Non-linear formats are not supported at all.
- Integer formats are prone to clipping/saturation.
- Floating formats may hit Windows peak protection, sounding awful.
In other words, it does not work properly and it never will. This will
be removed once minimum and maximum volumes are exposed properly.

As meager consolation, there are two advantages over endpoint volume:
- The code is much simpler.
- Other audio sessions are not affected.