Skip to content
GitLab
  • Menu
Projects Groups Snippets
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
  • Sign in / Register
  • chatty chatty
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Issues 263
    • Issues 263
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 15
    • Merge requests 15
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Packages & Registries
    • Packages & Registries
    • Container Registry
  • Monitor
    • Monitor
    • Incidents
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Repository
  • Wiki
    • Wiki
  • Snippets
    • Snippets
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • Librem5
  • chattychatty
  • Merge requests
  • !17

ChattyConversation: Be a bit more robust when parsing

  • Review changes

  • Download
  • Email patches
  • Plain diff
Merged Guido Gunther requested to merge guido.gunther/chatty:time into master Oct 21, 2018
  • Overview 0
  • Commits 2
  • Pipelines 4
  • Changes 1

Fix timestamp parsing:

This fixes 14:03:42 being displayed as 4:03:42

Also skip the opening brace in parse_message so we don't let the actual format leak into other functions.

Also split into two strings max so we don't lose parts of the message if it contains split chars.

Edited Oct 21, 2018 by Guido Gunther
Assignee
Assign to
Reviewer
Request review from
Time tracking
Source branch: time