A little cleanup

* Remove redundant get size method (inherited)
* Remove unnecessary super()

Author: kastman

nibabel/nifti1.py

@@ -393,8 +393,6 @@ def __init__(self, code, content):
         self._is_implicit_VR = self._guess_implicit_VR()
         self._content = self._unmangle(content)
 
-        super(Nifti1DicomExtension,self)
-
     def _guess_implicit_VR(self):
         """Without a DICOM Transfer Syntax, it's difficult to tell if Value
         Representations (VRs) are included in the DICOM encoding or not.
@@ -424,16 +422,6 @@ def _mangle(self, value):
         dio.seek(0)
         return dio.read(ds_len)
 
-    def get_sizeondisk(self):
-        """Return the size of the extension in the NIfTI file.
-        """
-        size = len(self._mangle(self._content))
-        size += 8
-        # extensions size has to be a multiple of 16 bytes
-        size += 16 - (size % 16)
-        
-        return size
-
 try:
     from dicom.filereader import read_dataset
     from dicom.filewriter import write_dataset

nibabel/tests/test_nifti1.py

@@ -1087,7 +1087,7 @@ def test_nifti_dicom_extension():
     hdr = nim.header
     exts_container = hdr.extensions
 
-    # create a single dicom tag (Patien ID, [0010,0020]) with Explicit VR
+    # create a single dicom tag (Patient ID, [0010,0020]) with Explicit VR
     dcmbytes_explicit = struct.pack('<HH2sH4s',0x10,0x20,
         'LO'.encode('utf-8'),4,'NiPy'.encode('utf-8'))
     dcmext = Nifti1DicomExtension(2,dcmbytes_explicit)
@@ -1099,7 +1099,7 @@ def test_nifti_dicom_extension():
     assert_equal(dcmext._mangle(dcmext.get_content()),dcmbytes_explicit)
     assert_equal(dcmext.get_sizeondisk() % 16, 0)
 
-    # create a single dicom tag (Patien ID, [0010,0020]) with Implicit VR
+    # create a single dicom tag (Patient ID, [0010,0020]) with Implicit VR
     dcmbytes_implicit = struct.pack('<HHL4s',0x10,0x20,4,'NiPy'.encode('utf-8'))
     dcmext = Nifti1DicomExtension(2,dcmbytes_implicit)
     assert_equal(dcmext._guess_implicit_VR(),True)