Remove duplicate definition of out_of_memory macro

[thatguystone]

When combining the project into a single file (as go-libsass does), gcc complains that out_of_memory is redefined.

Ref: wellington/go-libsass#57

[xzyfer]

Is this enough? The out_of_memory macro defined in util.cpp is under the Sass namespace

[nschonni]

This file is a copy/paste from another library, so editiing might not be the best solution. Just thinking of if we ever try to sync it again (although I think it's been static since the 2011 copyright header) ❤️

[thatguystone]

It looks like the file came from json.c, which, as @nschonni noted, hasn't been updated since 2011.

Apart from the usage in json.cpp, out_of_memory is only used once, so it might be better just inline it. Thoughts?

[xzyfer]

I'm ok with inlininh, or even just renaming the Sass namespaced version.

…

On Wed., 28 Mar. 2018, 5:45 am Andrew Stone, ***@***.***> wrote: It looks like the file came from json.c <https://github.com/rustyrussell/ccan/blob/master/ccan/json/json.c>, which, as @nschonni <https://github.com/nschonni> noted, hasn't been updated since 2011. Apart from the usage in json.cpp, out_of_memory is only used once, so it might be better just inline it. Thoughts? — You are receiving this because you commented. Reply to this email directly, view it on GitHub <#2619 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AAjZWMvJLbtKFbFIYlJqiXO1N2hTJKImks5tiojXgaJpZM4S8FZZ> .

[xzyfer]

Thanks