[OpenSmalltalk/opensmalltalk-vm] 47488a: Fix -textenc option parsing in unix vm

Previous Topic Next Topic
 
classic Classic list List threaded Threaded
1 message Options
Reply | Threaded
Open this post in threaded view
|

[OpenSmalltalk/opensmalltalk-vm] 47488a: Fix -textenc option parsing in unix vm

Eliot Miranda-3
 
  Branch: refs/heads/Cog
  Home:   https://github.com/OpenSmalltalk/opensmalltalk-vm
  Commit: 47488a8969c151b53f2cbd319d16894d44408474
      https://github.com/OpenSmalltalk/opensmalltalk-vm/commit/47488a8969c151b53f2cbd319d16894d44408474
  Author: Nicolas Cellier <[hidden email]>
  Date:   2018-01-03 (Wed, 03 Jan 2018)

  Changed paths:
    M platforms/unix/vm/sqUnixMain.c

  Log Message:
  -----------
  Fix -textenc option parsing in unix vm

The terminal null character of text encoding was not copied to the uppercase variant.
This is a bug, alloca does not set the allocated bytes to zero.

While at it, use VMOPTION macro where we handle vm option strings.
We may get rid of VMOPTION macro in the future, but this is another decision.


  Commit: 57154feca75e4e279c5b022bfffe98e4ebb9d5ab
      https://github.com/OpenSmalltalk/opensmalltalk-vm/commit/57154feca75e4e279c5b022bfffe98e4ebb9d5ab
  Author: Nicolas Cellier <[hidden email]>
  Date:   2018-01-03 (Wed, 03 Jan 2018)

  Changed paths:
    M platforms/unix/vm/sqUnixMain.c

  Log Message:
  -----------
  Merge pull request #182 from OpenSmalltalk/Fix_textenc_option_parsing_in_unix_vm

Fix -textenc option parsing in unix vm


Compare: https://github.com/OpenSmalltalk/opensmalltalk-vm/compare/8b1a98868722...57154feca75e