[Vm-dev] [OpenSmalltalk/opensmalltalk-vm] f1bc1c: Fix copy/paste typo when printing registers

Nicolas Cellier noreply at github.com
Wed Oct 23 16:23:26 UTC 2019


  Branch: refs/heads/Cog
  Home:   https://github.com/OpenSmalltalk/opensmalltalk-vm
  Commit: f1bc1cce999b3bcca317baa84bd0b28f21a4d7b0
      https://github.com/OpenSmalltalk/opensmalltalk-vm/commit/f1bc1cce999b3bcca317baa84bd0b28f21a4d7b0
  Author: Nicolas Cellier <nicolas.cellier.aka.nice at gmail.com>
  Date:   2019-10-20 (Sun, 20 Oct 2019)

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

  Log Message:
  -----------
  Fix copy/paste typo when printing registers


  Commit: 0e8de8580aedea1052fc771d6f301ba8fd17ef4e
      https://github.com/OpenSmalltalk/opensmalltalk-vm/commit/0e8de8580aedea1052fc771d6f301ba8fd17ef4e
  Author: Nicolas Cellier <nicolas.cellier.aka.nice at gmail.com>
  Date:   2019-10-20 (Sun, 20 Oct 2019)

  Changed paths:
    M platforms/unix/plugins/SocketPlugin/sqUnixSocket.c

  Log Message:
  -----------
  Remove a pointer aliasing (replace by memcpy).

Reason: gcc compiler barks.
platforms/unix/plugins/SocketPlugin/sqUnixSocket.c:1452:3: warning: dereferencing type-punned pointer will break strict-aliasing rules [-Wstrict-aliasing]
   *(int *)buf= 1;
It's probably benign in this case, but rather than trying to confirm this for all current and future versions of compilers, it's far easier to just avoid breaking strict aliasing.


  Commit: 86e976be8ca1a830d16c2e9f320258e6e69f291e
      https://github.com/OpenSmalltalk/opensmalltalk-vm/commit/86e976be8ca1a830d16c2e9f320258e6e69f291e
  Author: Nicolas Cellier <nicolas.cellier.aka.nice at gmail.com>
  Date:   2019-10-20 (Sun, 20 Oct 2019)

  Changed paths:
    M platforms/unix/vm-display-X11/sqUnixX11.c

  Log Message:
  -----------
  Remove a (false positive) -Wmaybe-uninitialized warning

Tell why we do it, why it's not necessary (as long as int have 32 bits).
Analyzing warnings again and again is time consuming.
So even false positive should better be eliminated.


  Commit: 09878ce6d3ae50385ecd63685fb486f4b925d7d8
      https://github.com/OpenSmalltalk/opensmalltalk-vm/commit/09878ce6d3ae50385ecd63685fb486f4b925d7d8
  Author: Nicolas Cellier <nicolas.cellier.aka.nice at gmail.com>
  Date:   2019-10-20 (Sun, 20 Oct 2019)

  Changed paths:
    M platforms/unix/vm-display-X11/sqUnixX11.c

  Log Message:
  -----------
  Remove two warnings about incompatible pointer type

Implementation should conform to SqDisplay.h

platforms/unix/vm-display-X11/sqUnixX11.c:7453:1: warning: incompatible pointer types initializing 'long (*)(char *, int)' with an expression of type 'long (char *, long)' [-Wincompatible-pointer-types]
SqDisplayDefine(X11);
^~~~~~~~~~~~~~~~~~~~
platforms/unix/vm/SqDisplay.h:108:3: note: expanded from macro 'SqDisplayDefine'
  display_winImageFind,                         \
  ^~~~~~~~~~~~~~~~~~~~
/platforms/unix/vm-display-X11/sqUnixX11.c:7453:1: warning: incompatible pointer types initializing 'long (*)(unsigned int *, long, long, long, long, long, long, long, long)' with an expression of type 'long (unsigned int *, long, long, long, int, int, int, int, int)' [-Wincompatible-pointer-types]
SqDisplayDefine(X11);
^~~~~~~~~~~~~~~~~~~~
platforms/unix/vm/SqDisplay.h:157:3: note: expanded from macro 'SqDisplayDefine'
  display_hostWindowShowDisplay,                \
  ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~


  Commit: 1c3a0ca9df711d31f6964a6511a2b7e98abab3e8
      https://github.com/OpenSmalltalk/opensmalltalk-vm/commit/1c3a0ca9df711d31f6964a6511a2b7e98abab3e8
  Author: Nicolas Cellier <nicolas.cellier.aka.nice at gmail.com>
  Date:   2019-10-21 (Mon, 21 Oct 2019)

  Changed paths:
    M platforms/unix/vm-display-null/sqUnixDisplayNull.c

  Log Message:
  -----------
  Same uncompatible pointer fix for display-null


  Commit: 622f8772df441d3d24ca34667c20f7eb9e2390c9
      https://github.com/OpenSmalltalk/opensmalltalk-vm/commit/622f8772df441d3d24ca34667c20f7eb9e2390c9
  Author: Nicolas Cellier <nicolas.cellier.aka.nice at gmail.com>
  Date:   2019-10-23 (Wed, 23 Oct 2019)

  Changed paths:
    M scripts/ci/travis_install.sh

  Log Message:
  -----------
  Attempt a blind fix against CI failure


  Commit: a4b8a0fe12ce85d263cffeb326a3306c990945db
      https://github.com/OpenSmalltalk/opensmalltalk-vm/commit/a4b8a0fe12ce85d263cffeb326a3306c990945db
  Author: Nicolas Cellier <nicolas.cellier.aka.nice at gmail.com>
  Date:   2019-10-23 (Wed, 23 Oct 2019)

  Changed paths:
    M platforms/unix/plugins/SocketPlugin/sqUnixSocket.c
    M platforms/unix/vm-display-X11/sqUnixX11.c
    M platforms/unix/vm-display-null/sqUnixDisplayNull.c
    M scripts/ci/travis_install.sh

  Log Message:
  -----------
  Merge branch 'small_fixes' into Cog


  Commit: 7875a84c55aaf2ab56ce4b242782b74d7c93d3c7
      https://github.com/OpenSmalltalk/opensmalltalk-vm/commit/7875a84c55aaf2ab56ce4b242782b74d7c93d3c7
  Author: Nicolas Cellier <nicolas.cellier.aka.nice at gmail.com>
  Date:   2019-10-23 (Wed, 23 Oct 2019)

  Changed paths:
    M .travis.yml

  Log Message:
  -----------
  Temporarily allow Newspeak failures

These allowances should be retracted once Newspeak builds are fixed.


Compare: https://github.com/OpenSmalltalk/opensmalltalk-vm/compare/ff593d12fa66...7875a84c55aa


More information about the Vm-dev mailing list