Hello there, I am writing internal docs for datasets.h and I noticed that while creating a new dataset, we start by Dataset *DatasetAlloc(const char *name); which only allocate memory via SCCalloc and then set the id of the struct, no mention whatsoever of *name. The only caller (DatasetGetOrCreate) of the former function is responsible of copying the name to the new dataset among other fields.
I just wanted to mention it before writing the note in datasets.h