Re: [PATCH] Introduce SAL_DELETE

Maybe give it a more explicit name?
Without looking, it's not clear at all what the macro does.

Perhaps
SAL_DELETE_MEMBER
or
SAL_CPP11_DELETE
??

On 2012-12-11 17:00, Stephan Bergmann (via Code Review) wrote:
> Hi,
>
> I have submitted a patch for review:
>
>      https://gerrit.libreoffice.org/1299
>
> To pull it, you can do:
>
>      git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/99/1299/1
>
> Introduce SAL_DELETE
>
> Change-Id: I7ac7392646ea34fb2e6a9f15990aca05a2c1e484
> ---
> M sal/inc/rtl/ustrbuf.hxx
> M sal/inc/sal/types.h
> M sfx2/inc/sfx2/docfile.hxx
> 3 files changed, 21 insertions(+), 4 deletions(-)
>
>
> --
> To view, visit https://gerrit.libreoffice.org/1299
> To unsubscribe, visit https://gerrit.libreoffice.org/settings
>
> Gerrit-MessageType: newchange
> Gerrit-Change-Id: I7ac7392646ea34fb2e6a9f15990aca05a2c1e484
> Gerrit-PatchSet: 1
> Gerrit-Project: core
> Gerrit-Branch: master
> Gerrit-Owner: Stephan Bergmann <sbergman@redhat.com>
>
> _______________________________________________
> LibreOffice mailing list
> LibreOffice@lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/libreoffice


Disclaimer: http://www.peralex.com/disclaimer.html