[DVDnav-discuss] r899 - trunk/libdvdnav/src/settings.c

[nicodvb]

Author: nicodvb
Date: Sun Apr 22 23:29:21 2007
New Revision: 899

Log:
removed more silly checks on !this before every public function

Modified:
   trunk/libdvdnav/src/settings.c

Modified: trunk/libdvdnav/src/settings.c
==============================================================================
--- trunk/libdvdnav/src/settings.c	(original)
+++ trunk/libdvdnav/src/settings.c	Sun Apr 22 23:29:21 2007
@@ -41,21 +41,11 @@
 /* Characteristics/setting API calls */
 
 dvdnav_status_t dvdnav_get_region_mask(dvdnav_t *this, int32_t *region) {
-  if(!this || !region) {
-    printerr("Passed a NULL this pointer.");
-    return DVDNAV_STATUS_ERR;
-  }
-
   (*region) = this->vm->state.registers.SPRM[20];
   return DVDNAV_STATUS_OK;
 }
 
 dvdnav_status_t dvdnav_set_region_mask(dvdnav_t *this, int32_t mask) {
-  if(!this) {
-    printerr("Passed a NULL this pointer.");
-    return DVDNAV_STATUS_ERR;
-  }
-
   pthread_mutex_lock(&this->vm_lock);
   this->vm->state.registers.SPRM[20] = (mask & 0xff);
   pthread_mutex_unlock(&this->vm_lock);
@@ -63,31 +53,16 @@ dvdnav_status_t dvdnav_set_region_mask(d
 }
 
 dvdnav_status_t dvdnav_set_readahead_flag(dvdnav_t *this, int32_t use_readahead) {
-  if(!this) {
-    printerr("Passed a NULL this pointer.");
-    return DVDNAV_STATUS_ERR;
-  }
-
   this->use_read_ahead = use_readahead;
   return DVDNAV_STATUS_OK;
 }
 
 dvdnav_status_t dvdnav_get_readahead_flag(dvdnav_t *this, int32_t *flag) {
-  if(!this || !flag) {
-    printerr("Passed a NULL this pointer.");
-    return DVDNAV_STATUS_ERR;
-  }
-
   (*flag) = this->use_read_ahead;
   return DVDNAV_STATUS_OK;
 }
 
 static dvdnav_status_t set_language_register(dvdnav_t *this, char *code, int reg) {
-  if(!this || !code) {
-    printerr("Passed a NULL this pointer.");
-    return DVDNAV_STATUS_ERR;
-  }
-    
   if(!code[0] || !code[1]) {
     printerr("Passed illegal language code.");
     return DVDNAV_STATUS_ERR;
@@ -112,21 +87,11 @@ dvdnav_status_t dvdnav_spu_language_sele
 }
 
 dvdnav_status_t dvdnav_set_PGC_positioning_flag(dvdnav_t *this, int32_t pgc) {
-  if(!this) {
-    printerr("Passed a NULL this pointer.");
-    return DVDNAV_STATUS_ERR;
-  }
-
   this->pgc_based = pgc;
   return DVDNAV_STATUS_OK;
 }
 
 dvdnav_status_t dvdnav_get_PGC_positioning_flag(dvdnav_t *this, int32_t *flag) {
-  if(!this || !flag) {
-    printerr("Passed a NULL this pointer.");
-    return DVDNAV_STATUS_ERR;
-  }
-
   (*flag) = this->pgc_based;
   return DVDNAV_STATUS_OK;
 }