dee

Compilation files with misleading-indentation

Bug #1579529 reported by Quick_Wango
14
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Telegram Desktop
Confirmed
Undecided
auchri
dee
New
Undecided
Unassigned

Bug Description

The compilation fails on my system (Archlinux) with a misleading-indentation warning (treated as an error) due to line 1329 in dee-serializable-model.c using GCC 6.1.1

The pos++ statement is probably supposed to be executed with every loop iteration and not just once after the loop.

See original description
Tags: compilation
Quick_Wango (quick-wango)
tags: added: compilation
Revision history for this message
Quick_Wango (quick-wango) wrote :

Here a patch that fixes the problem

Revision history for this message
auchri (auer-chrisi) wrote :

https://github.com/telegramdesktop/tdesktop/issues/2005

Changed in telegram-desktop:
assignee: nobody → Christoph Auer (auer-chrisi)
Revision history for this message
auchri (auer-chrisi) wrote :

@Quick_Wango Please release a new version with the patch asap.

Revision history for this message
Quick_Wango (quick-wango) wrote :

@auchri I'm not a maintainer. I just wrote the patch to be able to build Telegram locally and thought I'd publish it

Quick_Wango (quick-wango)
description: updated
Revision history for this message
Mikkel Kamstrup Erlandsen (kamstrup) wrote :

This looks like a bug. Can you add a regression test for it Quick-wango?

Revision history for this message
Quick_Wango (quick-wango) wrote :

@Mikkel actually no, I don't have much time at the moment. I just needed I fix for that and shared it. I also don't have much experience in C

Revision history for this message
Mikkel Kamstrup Erlandsen (kamstrup) wrote :

Ok, thanks for your contribution nonetheless :-) If I still have commit rights to this repo I might be able to do something ...

auchri (auer-chrisi)
Changed in telegram-desktop:
status: New → Confirmed
Revision history for this message
Michael Healy (horsemanoffaith-9) wrote :

I can confirm this bug in the dee package. It effects me on two separate Archlinux systems. Both fail to build without the patch, and builds successfully with the patch applied.

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.