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.

This reverts commit c4d74ccf.
This caused interferences with other audio sessions and violated the
audio volume guidelines from Microsoft.

Conflicts:
	modules/audio_output/mmdevice.c

Regression from 3044105e.

This gets quoted later, so don't quote it first.
Signed-off-by: Felix Paul Kühne <fkuehne@videolan.org>

This makes it a little more obvious, instead of just saying that tools
are missing or too old.