Introduce JERRY_DISABLE_HEAVY_DEBUG preprocessor definition.

[sand1k]

Introduce JERRY_HEAVY_DEBUG preprocessor definiton to speed up debug version of the engine.
JERRY_HEAVY_DEBUG is enabled only for unit tests.

Related issue: #180

[ruben-ayrapetyan]

@sand1k, could you, please, also add #ifdef JERRY_HEAVY_DEBUG to mem_check_heap and mem_check_pool?

[ruben-ayrapetyan]

Could you also, please, add check

#if defined (JERRY_NDEBUG) || !defined (JERRY_HEAVY_DEBUG)
# error "defined (JERRY_NDEBUG) || !defined (JERRY_HEAVY_DEBUG) in a unit test"
#endif /* JERRY_NDEBUG || !JERRY_HEAVY_DEBUG */

to test-common.h?

[egavrin]

@sand1k, could you, please, also add #ifdef JERRY_HEAVY_DEBUG to mem_check_heap and mem_check_pool?

+1

[ruben-ayrapetyan]

May be, we should rename JERRY_HEAVY_DEBUG to JERRY_NO_HEAVY_DEBUG (or something like this) to make it consistent with JERRY_NDEBUG.

What do you think about this?

[sand1k]

@ruben-ayrapetyan, name, consistent with JERRY_NDEBUG, would be JERRY_NHEAVY_DEBUG, but JERRY_NO_HEAVY_DEBUG looks much more clear for me. So my preference is JERRY_NO_HEAVY_DEBUG :)

[ruben-ayrapetyan]

@sand1k, great!

name, consistent with JERRY_NDEBUG, would be JERRY_NHEAVY_DEBUG

I mean consistency of behaviours =)

[sand1k]

@ruben-ayrapetyan, I've updated the pull request according to your notes.

[ruben-ayrapetyan]

Here we don't need !, as the corresponding code part is executed if the macro is defined.

[sand1k]

@ruben-ayrapetyan, @egavrin, pr was updated.

[sand1k]

@egavrin, please, review.
Should I rename JERRY_NDEBUG -> JERRY_DISABLE_DEBUG here (as a second commit) or create a separate PR?

[LaszloLango]

@sand1k, please do the JERRY_NDEBUG -> JERRY_DISABLE_DEBUG renaming in a separate PR. But why is it needed at all?

[sand1k]

@LaszloLango, just to make the definition name more clear.

[LaszloLango]

+1 LGTM

[zherczeg]

lgtm