Question

我想将此宏重写为c ++函数。

#define SAFE_BUFFER_DELETE(buf)     { if(buf != NULL) { buffer_delete(buf); buf = NULL; } }

附加信息http://paste.ubuntu.com/13940691/ abot .header文件

    void buffer_read(LPBUFFER buffer, void * buf, int bytes) {
        thecore_memcpy(buf, buffer->read_point, bytes);
        buffer_read_proceed(buffer, bytes);
    }

void buffer_delete(LPBUFFER buffer) {
    if (buffer == NULL) {
        return;
    }
    buffer_reset(buffer);

    int size = buffer->mem_size;

    int pool_index = buffer_get_exac_pool_index(size);
    if (pool_index >= 0) {
        BUFFER** buffer_pool = normalized_buffer_pool + pool_index;
        buffer->next = *buffer_pool;
        *buffer_pool = buffer;
    } else {
        free(buffer->mem_data);
        free(buffer);
    }
}

Answer 1

通过查看buffer_delete的源代码，显然if (buf != NULL)的检查完全是多余的。所以摆脱它。

完成后，宏会做两件事。

就其论点

buffer_delete

将NULL分配给其参数

如何处理函数中的第一件事是很清楚的。在C ++中，函数也可以接受参数作为引用，因此可以修改它。你的功能看起来像这样。

inline void
safe_delete_buffer(LPBUFFER& buf)
{
  buffer_delete(buf);
  buf = nullptr;
}

我假设LPBUFFER对于某些指针类型是typedef。

这种功能的行为对用户是否有用和直观是另一个问题。

将宏重写为c ++函数？

1 个答案: