Fix if-else syntax for the case missing OSS support #22

Sapludināts

SlavekB sapludināja 1 revīzijas no feat/fix-if-else-cond uz master

obache komentēja

Līdzstrādnieks

OSS MIDI API may be missing even if ALSA is enabled.

SlavekB recenzēja

SlavekB atstāja komentāru

Īpašnieks

Please consider simplifying the code as suggested in the comment.

libtdemid/deviceman.cc

													
					#endif

					#ifdef HAVE_OSS_SUPPORT

					    _seqbufptr=0;

					#endif

SlavekB komentēja

Īpašnieks

The combination of nested #ifdef and if-else in the code looks hard to read. I propose simplification as follows:

#ifdef HAVE_ALSA_SUPPORT
  if (alsa)
  {
    ((AlsaOut *)device[default_dev])->seqbuf_clean();
    return;
  }
#endif
#ifdef HAVE_OSS_SUPPORT
  {
    _seqbufptr=0;
   return;
  }
#endif

MicheleC komentēja

Īpašnieks

a simpler version is:

#ifdef HAVE_ALSA_SUPPORT
  if (alsa)
  {
    ((AlsaOut *)device[default_dev])->seqbuf_clean();
    return;
  }
#endif
#ifdef HAVE_OSS_SUPPORT
  _seqbufptr=0;
#endif

SlavekB komentēja

Īpašnieks

Yes, of course, this can make the code even simpler. I chose the closed block also for OSS to make it clear if there will be added support for some other sound systems – for example, for Solaris 😸

obache komentēja

Autors

Līdzstrādnieks

For multiple sound systems support, is it intend to be runtime switching or build time?

obache komentēja

Autors

Līdzstrādnieks

For multiple sound systems support, is it intend to be runtime switching or build time?

SlavekB komentēja

Īpašnieks

Please give attention to my review – see comment above.

obache komentēja

Autors

Līdzstrādnieks

Changed to coding style same as others.

SlavekB aizvēra šo izmaiņu pieprasījumu

SlavekB izdzēsa atzaru feat/fix-if-else-cond

SlavekB pievienoja atskaites punktu R14.0.6 release

Izmaiņu pieprasījums tika sapludināts ar revīziju dd6da7345a.

Pierakstieties, lai pievienotos šai sarunai.

Nav recenzentu

Nav atbildīgo

3 dalībnieki

Paziņojumi

Izpildes termiņš

Izpildes termiņš nav uzstādīts.

Atkarības

Nav atkarību.

Atsaucas uz: TDE/tdelibs#22