comparison dvdnav.c @ 264:8c86b61eb06f src

fix compiler warnings related to aliasing violations

263:d999338cab17

  /* 
   * user_ops_t is a structure of 32 bits.  We want to compute 
   * the union of two of those bitfields so to make this quicker 
   * than performing 32 ORs, we will access them as 32bits words.
   */
  uint32_t ops=0;
  
  if(!this) {
    printerr("Passed a NULL pointer.");
    return *(user_ops_t*)&ops;
  }
  if(!this->started) {
    printerr("Virtual DVD machine not started.");
    return *(user_ops_t*)&ops;
  }
  
  pthread_mutex_lock(&this->vm_lock); 
  ops|=*(uint32_t*)&this->pci.pci_gi.vobu_uop_ctl;
  
  if(this->vm && this->vm->state.pgc)
    ops|=*(uint32_t*)&this->vm->state.pgc->prohibited_ops;
  pthread_mutex_unlock(&this->vm_lock); 
    
  return *(user_ops_t*)&ops;
}

264:8c86b61eb06f

  /* 
   * user_ops_t is a structure of 32 bits.  We want to compute 
   * the union of two of those bitfields so to make this quicker 
   * than performing 32 ORs, we will access them as 32bits words.
   */
  union {
    user_ops_t ops_struct;
    uint32_t   ops_int;
  } ops;
  
  ops.ops_int = 0;
  
  if(!this) {
    printerr("Passed a NULL pointer.");
    return ops.ops_struct;
  }
  if(!this->started) {
    printerr("Virtual DVD machine not started.");
    return ops.ops_struct;
  }
  
  pthread_mutex_lock(&this->vm_lock); 
  ops.ops_int |= *(uint32_t*)&this->pci.pci_gi.vobu_uop_ctl;
  
  if(this->vm && this->vm->state.pgc)
    ops.ops_int |= *(uint32_t*)&this->vm->state.pgc->prohibited_ops;
  pthread_mutex_unlock(&this->vm_lock); 
    
  return ops.ops_struct;
}