Mixxx

Kill all c++ long variables from Mixxx Source

Bug #1094143 reported by Daniel Schürmann
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Mixxx
Fix Released
Low
Unassigned

Bug Description

On 64 bit architectures the standard c++ long has a different length across the OSs.

http://en.wikipedia.org/wiki/64-bit_architecture#64-bit_data_models

We may use:
http://doc.qt.digia.com/qt/qtglobal.html#qint64-typedef

This becomes an issue when we want to release an WIN64 build.

Tags: refactor
Revision history for this message
Daniel Schürmann (daschuer) wrote :

Not sure how to handle this bug? Do we expect issues wit it?

* Brute force: Use a refactoring tool to replace long with qint32
* Set a rule only for new code

Milestone?

Revision history for this message
RJ Skerry-Ryan (rryan) wrote :

1.12.0 is a good milestone target since it is a long way off so we can catch issues caused by this.

tags: added: refactor
Changed in mixxx:
status: New → Confirmed
importance: Undecided → Low
Revision history for this message
Daniel Schürmann (daschuer) wrote :

I think the work is done, no need to keep this bug open.

Changed in mixxx:
status: Confirmed → Fix Released
Revision history for this message
Swiftb0y (swiftb0y) wrote :

Mixxx now uses GitHub for bug tracking. This bug has been migrated to:
https://github.com/mixxxdj/mixxx/issues/6812

lock status: Metadata changes locked and limited to project staff
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.