[ovs-git] [openvswitch/ovs] 828129: json: Avoid extra memory allocation and string cop...

GitHub noreply at github.com
Sat Mar 31 19:36:37 UTC 2018


  Branch: refs/heads/master
  Home:   https://github.com/openvswitch/ovs
  Commit: 828129d9271da18c2742648aa910e1fe399bb8db
      https://github.com/openvswitch/ovs/commit/828129d9271da18c2742648aa910e1fe399bb8db
  Author: Ben Pfaff <blp at ovn.org>
  Date:   2018-03-31 (Sat, 31 Mar 2018)

  Changed paths:
    M include/openvswitch/json.h
    M include/openvswitch/shash.h
    M lib/json.c
    M lib/shash.c

  Log Message:
  -----------
  json: Avoid extra memory allocation and string copy parsing object members.

Until now, every time the JSON parser added an object member, it made an
extra copy of the member name and then freed the original copy.  This is
wasteful, so this commit eliminates the extra copy.

Signed-off-by: Ben Pfaff <blp at ovn.org>
Reviewed-by: Yifeng Sun <pkusunyifeng at gmail.com>


  Commit: bbb2cb209a76e7640ba93b91a0cbbe955fb16056
      https://github.com/openvswitch/ovs/commit/bbb2cb209a76e7640ba93b91a0cbbe955fb16056
  Author: Ben Pfaff <blp at ovn.org>
  Date:   2018-03-31 (Sat, 31 Mar 2018)

  Changed paths:
    M utilities/checkpatch.py

  Log Message:
  -----------
  checkpatch: Avoid warnings for /* or */.

checkpatch would sometimes confuse comment markers for operators.  This
fixes the problem.

Signed-off-by: Ben Pfaff <blp at ovn.org>
Acked-by: Aaron Conole <aconole at redhat.com>


Compare: https://github.com/openvswitch/ovs/compare/056caea89bff...bbb2cb209a76


More information about the git mailing list