obitools/obitools3
2
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases 66 Wiki Activity

Removed deprecated TODOs

Browse Source
This commit is contained in:
Celine Mercier
2016-12-19 14:36:40 +01:00
parent 3bf5260174
commit 9c71b06117
1 changed files with 3 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
python/obitools3/obidms/_obidmscolumn_str.pyx
View File

@ -22,7 +22,7 @@ cdef class OBIDMS_column_str(OBIDMS_column):
result = None result = None
else : else :
result = bytes2str(value) result = bytes2str(value)
# NOTE: value is not freed because the pointer points to a mmapped region in an AVL data file. (TODO discuss) # NOTE: value is not freed because the pointer points to a mmapped region in an AVL data file.
return result return result
cpdef set_line(self, index_t line_nb, object value): cpdef set_line(self, index_t line_nb, object value):
@ -46,7 +46,7 @@ cdef class OBIDMS_column_multi_elts_str(OBIDMS_column_multi_elts):
result = None result = None
else : else :
result = bytes2str(value) result = bytes2str(value)
# NOTE: value is not freed because the pointer points to a mmapped region in an AVL data file. (TODO discuss) # NOTE: value is not freed because the pointer points to a mmapped region in an AVL data file.
return result return result
cpdef object get_line(self, index_t line_nb) : cpdef object get_line(self, index_t line_nb) :
@ -65,7 +65,7 @@ cdef class OBIDMS_column_multi_elts_str(OBIDMS_column_multi_elts):
value_in_result = None value_in_result = None
else : else :
value_in_result = bytes2str(value) value_in_result = bytes2str(value)
# NOTE: value is not freed because the pointer points to a mmapped region in an AVL data file. (TODO discuss) # NOTE: value is not freed because the pointer points to a mmapped region in an AVL data file.
result[self.elements_names[i]] = value_in_result result[self.elements_names[i]] = value_in_result
if all_NA and (value_in_result is not None) : if all_NA and (value_in_result is not None) :
all_NA = False all_NA = False