Update TalkatuMessage:id to be a string.

Review Request #879 — Created Aug. 5, 2021 and submitted

Information

talkatu/talkatu
default
49e874464f76

Reviewers

Many modern protocols are using uuids as id's and forcing that into a uint64 is
going to lead to collisions. So instead just use a string which should work for
everything.

Ran the demo and unit tests.

Description From Last Updated

This should be take_string, since get_id is not const.

QuLogicQuLogic
QuLogic
  1. Ship It!
  2. 
      
QuLogic
  1. 
      
  2. talkatu/talkatuinput.c (Diff revision 1)
     
     
    Show all issues

    This should be take_string, since get_id is not const.

    1. Oops, no, that's not TalkatuMessage, never mind. I think I mixed them up.

    2. cool 👍

  3. 
      
grim
Review request changed
Status:
Completed