[redland-dev] [Redland librdf RDF API 0000391]: [patch] librdf_free_foo(0)

Mantis Bug Tracker mantis-bug-sender at librdf.org
Tue Oct 12 10:56:26 CEST 2010


The following issue has been SUBMITTED. 
====================================================================== 
http://bugs.librdf.org/mantis/view.php?id=391 
====================================================================== 
Reported By:                mst
Assigned To:                
====================================================================== 
Project:                    Redland librdf RDF API
Issue ID:                   391
Category:                   api
Reproducibility:            always
Severity:                   minor
Priority:                   normal
Status:                     new
====================================================================== 
Date Submitted:             2010-10-12 08:56
Last Modified:              2010-10-12 08:56
====================================================================== 
Summary:                    [patch] librdf_free_foo(0)
Description: 
originally reported here:
http://lists.usefulinc.com/pipermail/redland-dev/2008-April/001738.html

since that page for some reason displays only an unhelpful base64 blob,
here is the text of the mail:

why do the librdf destructors not handle 0 arguments correctly?
i.e. they should just return, and not segfault like they currently do.
after all, the C free function and the C++ operator delete handle 0 
arguments the same, so i would guess that many people would implictily 
assume that anything that frees stuff acts the same way.

so, since writing this patch was less work than fixing the >2k lines of 
code that i wrote last week, here is a patch that fixes all the publicly 
documented librdf_free_foo functions.
will this go in the next release?
====================================================================== 

Issue History 
Date Modified    Username       Field                    Change               
====================================================================== 
2010-10-12 08:56 mst            New Issue                                    
2010-10-12 08:56 mst            File Added: redland-1.0.7-free.patch            
       
======================================================================



More information about the redland-dev mailing list