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 293
    • Issues 293
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 13
    • Merge requests 13
  • 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

Due to an influx of spam, we have had to impose restrictions on new accounts. Please see this wiki page for instructions on how to get full permissions. Sorry for the inconvenience.

  • Librem5
  • chattychatty
  • Merge requests
  • !14

conversation: Don't fail to display the newest messages

  • Review changes

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

When loading logs chatty would display the 50 oldest messages while we want the newest ones. Fix this by not reversing the history list and then prepending to the messges list. This is also faster, see https://github.com/GNOME/glib/blob/master/glib/glist.c#L237

Closes: #34 (closed)

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