[riot-commits] [RIOT-OS/RIOT] 0a4ea0: sys: use typedef for `struct bloom_t`

Christian Mehlis mehlis at inf.fu-berlin.de
Fri Sep 19 13:18:42 CEST 2014


  Branch: refs/heads/master
  Home:   https://github.com/RIOT-OS/RIOT
  Commit: 0a4ea07daa0691043712de61f0543126ab0a21b9
      https://github.com/RIOT-OS/RIOT/commit/0a4ea07daa0691043712de61f0543126ab0a21b9
  Author: Cenk Gündoğan <cnkgndgn at gmail.com>
  Date:   2014-09-17 (Wed, 17 Sep 2014)

  Changed paths:
    M sys/bloom/bloom.c
    M sys/include/bloom.h
    M tests/bloom/main.c
    M tests/bloom_bytes/main.c

  Log Message:
  -----------
  sys: use typedef for `struct bloom_t`
`bloom_t` is defined as a struct.
`_t` can mislead the user to think of bloom_t
as a typedef (see our coding conventions) instead of a struct.
Thus, I modified `struct bloom_t` to be a *typedefed* struct.

Another solution would be to rename bloom_t to sth. like bloom_s
everywhere and use `struct bloom_s` instead of `bloom_t`.


  Commit: 88ae531ef946aa83a06ce133ff859291fc506f0d
      https://github.com/RIOT-OS/RIOT/commit/88ae531ef946aa83a06ce133ff859291fc506f0d
  Author: Christian Mehlis <mehlis at inf.fu-berlin.de>
  Date:   2014-09-19 (Fri, 19 Sep 2014)

  Changed paths:
    M sys/bloom/bloom.c
    M sys/include/bloom.h
    M tests/bloom/main.c
    M tests/bloom_bytes/main.c

  Log Message:
  -----------
  Merge pull request #1679 from cgundogan/bloom_t_as_typedef

sys: use typedef for `struct bloom_t`


Compare: https://github.com/RIOT-OS/RIOT/compare/283076880b51...88ae531ef946


More information about the commits mailing list