Branch: refs/heads/master
  Home:   https://github.com/synfig/synfig
  Commit: 4b6602f4f7d86f3460f131f01b7d02a12b1af098
      
https://github.com/synfig/synfig/commit/4b6602f4f7d86f3460f131f01b7d02a12b1af098
  Author: Rodolfo Ribeiro Gomes <rodolf...@gmail.com>
  Date:   2021-07-05 (Mon, 05 Jul 2021)

  Changed paths:
    M synfig-studio/src/synfigapp/value_desc.h

  Log Message:
  -----------
  synfigapp: prefer nullptr rather 0 or NULL


  Commit: f27da8975d7eec3e1bc9d1b88f2ae9b7e7d82859
      
https://github.com/synfig/synfig/commit/f27da8975d7eec3e1bc9d1b88f2ae9b7e7d82859
  Author: Rodolfo Ribeiro Gomes <rodolf...@gmail.com>
  Date:   2021-07-05 (Mon, 05 Jul 2021)

  Changed paths:
    M synfig-studio/src/synfigapp/value_desc.h

  Log Message:
  -----------
  synfigapp: ValueDesc: don't make me try to remember magic numbers


  Commit: 7055eb5cc5868b9707fcf027acc5521e65cdac4f
      
https://github.com/synfig/synfig/commit/7055eb5cc5868b9707fcf027acc5521e65cdac4f
  Author: Rodolfo Ribeiro Gomes <rodolf...@gmail.com>
  Date:   2021-07-05 (Mon, 05 Jul 2021)

  Changed paths:
    M synfig-studio/src/synfigapp/value_desc.h

  Log Message:
  -----------
  synfigapp: ValueDesc: don't cast when it isn't necessary


  Commit: 5032236311e340a09755e3b1677809743b4fee06
      
https://github.com/synfig/synfig/commit/5032236311e340a09755e3b1677809743b4fee06
  Author: Rodolfo Ribeiro Gomes <rodolf...@gmail.com>
  Date:   2021-07-05 (Mon, 05 Jul 2021)

  Changed paths:
    M synfig-studio/src/synfigapp/value_desc.h

  Log Message:
  -----------
  synfigapp: ValueDesc: remove unused constructors

one is commented-out since 2005


  Commit: 1f083b22717caf671803db99fc41758173ef06e4
      
https://github.com/synfig/synfig/commit/1f083b22717caf671803db99fc41758173ef06e4
  Author: Rodolfo Ribeiro Gomes <rodolf...@gmail.com>
  Date:   2021-07-05 (Mon, 05 Jul 2021)

  Changed paths:
    M synfig-studio/src/synfigapp/value_desc.h

  Log Message:
  -----------
  synfigapp: ValueDesc: remove scalar: it's always 1.0

It's only valid for ValueDesc for LinkableValueNode, and it
is never set besides in constructor, where its values is set to 1.0


  Commit: 9568797e50499bc7b541df6d1210075d70f7cd17
      
https://github.com/synfig/synfig/commit/9568797e50499bc7b541df6d1210075d70f7cd17
  Author: Rodolfo Ribeiro Gomes <rodolf...@gmail.com>
  Date:   2021-07-05 (Mon, 05 Jul 2021)

  Changed paths:
    M synfig-studio/src/synfigapp/actions/valuedesclink.cpp

  Log Message:
  -----------
  ValueDescLink: link_scalar is always 1.0

Set link_scalar value:
- if ValueDesc is Linkable valuenode, ValueDesc::get_scalar returns 1.0
- otherwise, 1.0


  Commit: 0edff44caab07d0681174b5ab949e030ff13d9e7
      
https://github.com/synfig/synfig/commit/0edff44caab07d0681174b5ab949e030ff13d9e7
  Author: Rodolfo Ribeiro Gomes <rodolf...@gmail.com>
  Date:   2021-07-05 (Mon, 05 Jul 2021)

  Changed paths:
    M synfig-studio/src/synfigapp/actions/valuedesclink.cpp
    M synfig-studio/src/synfigapp/actions/valuedesclink.h
    M synfig-studio/src/synfigapp/value_desc.h

  Log Message:
  -----------
  ValueDescLink: if link_scalar is 0.0 or 1.0, this comparison is always false

ValueDesc::get_scalar() returns 1.0

ValueDesc::get_scalar() * 0.0 or 1.0 is always >= 0

So, the comparison is useless, also link_scalar


  Commit: 4a78b7335c02c54bb597dd6cec88c7a24beff8e4
      
https://github.com/synfig/synfig/commit/4a78b7335c02c54bb597dd6cec88c7a24beff8e4
  Author: Rodolfo Ribeiro Gomes <rodolf...@gmail.com>
  Date:   2021-07-05 (Mon, 05 Jul 2021)

  Changed paths:
    M synfig-studio/src/synfigapp/actions/valuedesclink.cpp
    M synfig-studio/src/synfigapp/value_desc.h

  Log Message:
  -----------
  ValueDescLink: remove useless code

if(iter_scalar*current_scalar < 0) is always false because
get_scalar() used to be 0,0 or 1.0 only

So, the comparison is useless and so this whole section as
it would only change the found_inverse to true - but it would
never actually do it.

As this was the last code that would use ValueDesc::get_scalar(),
this method can be safely removed.


  Commit: a65fcc53f0e05410abc1989ba7102cb492fc71e5
      
https://github.com/synfig/synfig/commit/a65fcc53f0e05410abc1989ba7102cb492fc71e5
  Author: Rodolfo Ribeiro Gomes <rodolf...@gmail.com>
  Date:   2021-07-05 (Mon, 05 Jul 2021)

  Changed paths:
    M synfig-studio/src/synfigapp/actions/valuedesclink.cpp

  Log Message:
  -----------
  ValueDescLink: remove useless code: found_inverse is always false


  Commit: c0409aad967caacc1e115d47f391aecee88eed72
      
https://github.com/synfig/synfig/commit/c0409aad967caacc1e115d47f391aecee88eed72
  Author: Rodolfo Ribeiro Gomes <rodolf...@gmail.com>
  Date:   2021-07-05 (Mon, 05 Jul 2021)

  Changed paths:
    M synfig-studio/src/synfigapp/actions/valuedesclink.cpp

  Log Message:
  -----------
  ValueDescLink: reduce variable scope and use for-ranged loop


  Commit: 9acfd9c0e1b42b5bdf6a31747930d2be855224ee
      
https://github.com/synfig/synfig/commit/9acfd9c0e1b42b5bdf6a31747930d2be855224ee
  Author: ice0 <konop...@gmail.com>
  Date:   2021-07-05 (Mon, 05 Jul 2021)

  Changed paths:
    M synfig-studio/src/synfigapp/actions/valuedesclink.cpp
    M synfig-studio/src/synfigapp/actions/valuedesclink.h
    M synfig-studio/src/synfigapp/value_desc.h

  Log Message:
  -----------
  Merge PR #2206: Clean synfigapp::ValueDesc a bit

Clean synfigapp::ValueDesc a bit


Compare: https://github.com/synfig/synfig/compare/2c2062c5d115...9acfd9c0e1b4


_______________________________________________
Synfig-devl mailing list
Synfig-devl@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/synfig-devl

Reply via email to