Skip to content

Commit

Permalink
- When importing from standard MIDI files (SMF), set track
Browse files Browse the repository at this point in the history
  names from Mtrk TRACKNAME meta-events instead of filename.
  • Loading branch information
rncbc committed Feb 20, 2018
1 parent fdd5dec commit ae124c3
Show file tree
Hide file tree
Showing 11 changed files with 51 additions and 44 deletions.
3 changes: 3 additions & 0 deletions ChangeLog
Expand Up @@ -6,6 +6,9 @@ ChangeLog

GIT HEAD

- When importing from standard MIDI files (SMF), set track
names from Mtrk TRACKNAME meta-events instead of filename.

- Avoid asking to save as to existing or just newly created
clip file-names, whenever possible. (EXPERIMENTAL)

Expand Down
10 changes: 7 additions & 3 deletions src/qtractorMidiClip.cpp
@@ -1,7 +1,7 @@
// qtractorMidiClip.cpp
//
/****************************************************************************
Copyright (C) 2005-2017, rncbc aka Rui Nuno Capela. All rights reserved.
Copyright (C) 2005-2018, rncbc aka Rui Nuno Capela. All rights reserved.
This program is free software; you can redistribute it and/or
modify it under the terms of the GNU General Public License
Expand Down Expand Up @@ -476,8 +476,12 @@ bool qtractorMidiClip::openMidiFile (
// We should have events, otherwise this clip is of no use...
//if (m_pSeq->events().count() < 1)
// return false;
// And initial clip name...
pSeq->setName(shortClipName(QFileInfo(m_pFile->filename()).baseName()));
// And initial clip name,
// if not already set from SMF TRACKNAME meta-event...
if (pSeq->name().isEmpty()) {
pSeq->setName(shortClipName(
QFileInfo(m_pFile->filename()).baseName()));
}
}

// Actual track-channel is set by now...
Expand Down
2 changes: 1 addition & 1 deletion src/qtractorMidiClip.h
@@ -1,7 +1,7 @@
// qtractorMidiClip.h
//
/****************************************************************************
Copyright (C) 2005-2017, rncbc aka Rui Nuno Capela. All rights reserved.
Copyright (C) 2005-2018, rncbc aka Rui Nuno Capela. All rights reserved.
This program is free software; you can redistribute it and/or
modify it under the terms of the GNU General Public License
Expand Down
10 changes: 5 additions & 5 deletions src/translations/qtractor_cs.ts
Expand Up @@ -152,31 +152,31 @@ Konec: %2 Délka: %3</translation>
<translation>%1 - Banka %2</translation>
</message>
<message>
<location filename="../qtractorMidiClip.cpp" line="1216"/>
<location filename="../qtractorMidiClip.cpp" line="1220"/>
<source>(format %1)
MIDI: </source>
<translation>(formát %1)
MIDI: </translation>
</message>
<message>
<location filename="../qtractorMidiClip.cpp" line="1218"/>
<location filename="../qtractorMidiClip.cpp" line="1222"/>
<location filename="../qtractorMidiListView.cpp" line="73"/>
<source>Channel %1</source>
<translation>Kanál %1</translation>
</message>
<message>
<location filename="../qtractorMidiClip.cpp" line="1220"/>
<location filename="../qtractorMidiClip.cpp" line="1224"/>
<location filename="../qtractorMidiListView.cpp" line="67"/>
<source>Track %1</source>
<translation>Stopa %1</translation>
</message>
<message>
<location filename="../qtractorMidiClip.cpp" line="1223"/>
<location filename="../qtractorMidiClip.cpp" line="1227"/>
<source>, %1 tracks, %2 tpqn</source>
<translation>, %1 stopy, %2 tpqn</translation>
</message>
<message>
<location filename="../qtractorMidiClip.cpp" line="1230"/>
<location filename="../qtractorMidiClip.cpp" line="1234"/>
<source> (%1% vol)</source>
<translation> (%1% hlasitost)</translation>
</message>
Expand Down
10 changes: 5 additions & 5 deletions src/translations/qtractor_de.ts
Expand Up @@ -316,31 +316,31 @@ End: %3 Length: %4</source>
<translation type="unfinished"></translation>
</message>
<message>
<location filename="../qtractorMidiClip.cpp" line="1216"/>
<location filename="../qtractorMidiClip.cpp" line="1220"/>
<source>(format %1)
MIDI: </source>
<translation>(Format %1)
MIDI: </translation>
</message>
<message>
<location filename="../qtractorMidiClip.cpp" line="1218"/>
<location filename="../qtractorMidiClip.cpp" line="1222"/>
<location filename="../qtractorMidiListView.cpp" line="73"/>
<source>Channel %1</source>
<translation>Kanal %1</translation>
</message>
<message>
<location filename="../qtractorMidiClip.cpp" line="1220"/>
<location filename="../qtractorMidiClip.cpp" line="1224"/>
<location filename="../qtractorMidiListView.cpp" line="67"/>
<source>Track %1</source>
<translation>Spur %1</translation>
</message>
<message>
<location filename="../qtractorMidiClip.cpp" line="1223"/>
<location filename="../qtractorMidiClip.cpp" line="1227"/>
<source>, %1 tracks, %2 tpqn</source>
<translation type="unfinished"></translation>
</message>
<message>
<location filename="../qtractorMidiClip.cpp" line="1230"/>
<location filename="../qtractorMidiClip.cpp" line="1234"/>
<source> (%1% vol)</source>
<translation type="unfinished"></translation>
</message>
Expand Down
10 changes: 5 additions & 5 deletions src/translations/qtractor_es.ts
Expand Up @@ -384,31 +384,31 @@ Fin: %3 Duración: %4</translation>
<translation>%1 - Banco %2</translation>
</message>
<message>
<location filename="../qtractorMidiClip.cpp" line="1216"/>
<location filename="../qtractorMidiClip.cpp" line="1220"/>
<source>(format %1)
MIDI: </source>
<translation>(formato %1)
MIDI: </translation>
</message>
<message>
<location filename="../qtractorMidiClip.cpp" line="1218"/>
<location filename="../qtractorMidiClip.cpp" line="1222"/>
<location filename="../qtractorMidiListView.cpp" line="73"/>
<source>Channel %1</source>
<translation>Canal %1</translation>
</message>
<message>
<location filename="../qtractorMidiClip.cpp" line="1220"/>
<location filename="../qtractorMidiClip.cpp" line="1224"/>
<location filename="../qtractorMidiListView.cpp" line="67"/>
<source>Track %1</source>
<translation>Pista %1</translation>
</message>
<message>
<location filename="../qtractorMidiClip.cpp" line="1223"/>
<location filename="../qtractorMidiClip.cpp" line="1227"/>
<source>, %1 tracks, %2 tpqn</source>
<translation>, %1 pistas, %2 tpqn</translation>
</message>
<message>
<location filename="../qtractorMidiClip.cpp" line="1230"/>
<location filename="../qtractorMidiClip.cpp" line="1234"/>
<source> (%1% vol)</source>
<translation> (%1% vol)</translation>
</message>
Expand Down
10 changes: 5 additions & 5 deletions src/translations/qtractor_fr.ts
Expand Up @@ -317,31 +317,31 @@ Fin : %3 Longueur : %4</translation>
<translation>%1 - Banque %2</translation>
</message>
<message>
<location filename="../qtractorMidiClip.cpp" line="1216"/>
<location filename="../qtractorMidiClip.cpp" line="1220"/>
<source>(format %1)
MIDI: </source>
<translation>(format %1)
MIDI: </translation>
</message>
<message>
<location filename="../qtractorMidiClip.cpp" line="1218"/>
<location filename="../qtractorMidiClip.cpp" line="1222"/>
<location filename="../qtractorMidiListView.cpp" line="73"/>
<source>Channel %1</source>
<translation>Canal %1</translation>
</message>
<message>
<location filename="../qtractorMidiClip.cpp" line="1220"/>
<location filename="../qtractorMidiClip.cpp" line="1224"/>
<location filename="../qtractorMidiListView.cpp" line="67"/>
<source>Track %1</source>
<translation>Piste %1</translation>
</message>
<message>
<location filename="../qtractorMidiClip.cpp" line="1223"/>
<location filename="../qtractorMidiClip.cpp" line="1227"/>
<source>, %1 tracks, %2 tpqn</source>
<translation>, %1 pistes, %2 tpqn</translation>
</message>
<message>
<location filename="../qtractorMidiClip.cpp" line="1230"/>
<location filename="../qtractorMidiClip.cpp" line="1234"/>
<source> (%1% vol)</source>
<translation> (%1% vol)</translation>
</message>
Expand Down
10 changes: 5 additions & 5 deletions src/translations/qtractor_it.ts
Expand Up @@ -318,31 +318,31 @@ Fine: %3 Lunghezza: %4</translation>
<translation>%1 - Banco %2</translation>
</message>
<message>
<location filename="../qtractorMidiClip.cpp" line="1216"/>
<location filename="../qtractorMidiClip.cpp" line="1220"/>
<source>(format %1)
MIDI: </source>
<translation>(formato %1)
MIDI: </translation>
</message>
<message>
<location filename="../qtractorMidiClip.cpp" line="1218"/>
<location filename="../qtractorMidiClip.cpp" line="1222"/>
<location filename="../qtractorMidiListView.cpp" line="73"/>
<source>Channel %1</source>
<translation>Canale %1</translation>
</message>
<message>
<location filename="../qtractorMidiClip.cpp" line="1220"/>
<location filename="../qtractorMidiClip.cpp" line="1224"/>
<location filename="../qtractorMidiListView.cpp" line="67"/>
<source>Track %1</source>
<translation>Traccia %1</translation>
</message>
<message>
<location filename="../qtractorMidiClip.cpp" line="1223"/>
<location filename="../qtractorMidiClip.cpp" line="1227"/>
<source>, %1 tracks, %2 tpqn</source>
<translation>, %1 tracce, %2 tpqn</translation>
</message>
<message>
<location filename="../qtractorMidiClip.cpp" line="1230"/>
<location filename="../qtractorMidiClip.cpp" line="1234"/>
<source> (%1% vol)</source>
<translation> (%1% vol)</translation>
</message>
Expand Down
10 changes: 5 additions & 5 deletions src/translations/qtractor_ja.ts
Expand Up @@ -316,31 +316,31 @@ End: %3 Length: %4</source>
<translation>%1 - バンク %2</translation>
</message>
<message>
<location filename="../qtractorMidiClip.cpp" line="1216"/>
<location filename="../qtractorMidiClip.cpp" line="1220"/>
<source>(format %1)
MIDI: </source>
<translation>(フォーマット %1)
MIDI: </translation>
</message>
<message>
<location filename="../qtractorMidiClip.cpp" line="1218"/>
<location filename="../qtractorMidiClip.cpp" line="1222"/>
<location filename="../qtractorMidiListView.cpp" line="73"/>
<source>Channel %1</source>
<translation>チャンネル %1</translation>
</message>
<message>
<location filename="../qtractorMidiClip.cpp" line="1220"/>
<location filename="../qtractorMidiClip.cpp" line="1224"/>
<location filename="../qtractorMidiListView.cpp" line="67"/>
<source>Track %1</source>
<translation>トラック %1</translation>
</message>
<message>
<location filename="../qtractorMidiClip.cpp" line="1223"/>
<location filename="../qtractorMidiClip.cpp" line="1227"/>
<source>, %1 tracks, %2 tpqn</source>
<translation>, %1 トラック、 %2 tpqn</translation>
</message>
<message>
<location filename="../qtractorMidiClip.cpp" line="1230"/>
<location filename="../qtractorMidiClip.cpp" line="1234"/>
<source> (%1% vol)</source>
<translation> (%1% ボリューム)</translation>
</message>
Expand Down
10 changes: 5 additions & 5 deletions src/translations/qtractor_pt.ts
Expand Up @@ -152,7 +152,7 @@ Fim: %3 Tamanho: %4 </translation>
<translation>Mostrar ajuda sobre opções em linha de comando</translation>
</message>
<message>
<location filename="../qtractorMidiClip.cpp" line="1230"/>
<location filename="../qtractorMidiClip.cpp" line="1234"/>
<source> (%1% vol)</source>
<translation> (%1% vol)</translation>
</message>
Expand Down Expand Up @@ -251,7 +251,7 @@ Fim: %3 Tamanho: %4 </translation>
<translation>Uso: %1 [opções] [arquivo-da-sessão]</translation>
</message>
<message>
<location filename="../qtractorMidiClip.cpp" line="1218"/>
<location filename="../qtractorMidiClip.cpp" line="1222"/>
<location filename="../qtractorMidiListView.cpp" line="73"/>
<source>Channel %1</source>
<translation>Canal %1</translation>
Expand Down Expand Up @@ -281,7 +281,7 @@ Fim: %3 Tamanho: %4 </translation>
<translation>%1 Gravar</translation>
</message>
<message>
<location filename="../qtractorMidiClip.cpp" line="1216"/>
<location filename="../qtractorMidiClip.cpp" line="1220"/>
<source>(format %1)
MIDI: </source>
<translation>(formato %1)
Expand Down Expand Up @@ -498,7 +498,7 @@ MIDI: </translation>
(%1 mudança de altura em semitons)</translation>
</message>
<message>
<location filename="../qtractorMidiClip.cpp" line="1220"/>
<location filename="../qtractorMidiClip.cpp" line="1224"/>
<location filename="../qtractorMidiListView.cpp" line="67"/>
<source>Track %1</source>
<translation>Pista %1</translation>
Expand Down Expand Up @@ -700,7 +700,7 @@ Pista: &quot;%1&quot; Entrada: &quot;%2&quot; Saída: &quot;%3&quot;</translatio
<translation>nó de atualização de tempo</translation>
</message>
<message>
<location filename="../qtractorMidiClip.cpp" line="1223"/>
<location filename="../qtractorMidiClip.cpp" line="1227"/>
<source>, %1 tracks, %2 tpqn</source>
<translation>, %1 pistas, %2 tpqn</translation>
</message>
Expand Down
10 changes: 5 additions & 5 deletions src/translations/qtractor_ru.ts
Expand Up @@ -150,31 +150,31 @@ End: %3 Length: %4</source>
<translation>%1 - Банк %2</translation>
</message>
<message>
<location filename="../qtractorMidiClip.cpp" line="1216"/>
<location filename="../qtractorMidiClip.cpp" line="1220"/>
<source>(format %1)
MIDI: </source>
<translation>(формат %1)
MIDI: </translation>
</message>
<message>
<location filename="../qtractorMidiClip.cpp" line="1218"/>
<location filename="../qtractorMidiClip.cpp" line="1222"/>
<location filename="../qtractorMidiListView.cpp" line="73"/>
<source>Channel %1</source>
<translation>Канал %1</translation>
</message>
<message>
<location filename="../qtractorMidiClip.cpp" line="1220"/>
<location filename="../qtractorMidiClip.cpp" line="1224"/>
<location filename="../qtractorMidiListView.cpp" line="67"/>
<source>Track %1</source>
<translation>Дорожка %1</translation>
</message>
<message>
<location filename="../qtractorMidiClip.cpp" line="1223"/>
<location filename="../qtractorMidiClip.cpp" line="1227"/>
<source>, %1 tracks, %2 tpqn</source>
<translation>, %1 дорожек, %2 tpqn</translation>
</message>
<message>
<location filename="../qtractorMidiClip.cpp" line="1230"/>
<location filename="../qtractorMidiClip.cpp" line="1234"/>
<source> (%1% vol)</source>
<translation> (%1% громк)</translation>
</message>
Expand Down

4 comments on commit ae124c3

@schnitzeltony
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This one might need a tiny Midi-Import extension update (text in configuration dialog?). Can you merge midiimportx so it's easier for me to test and I'll see what's necessary.

@rncbc
Copy link
Owner Author

@rncbc rncbc commented on ae124c3 Feb 22, 2018

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

already merged in d0efcec !

@schnitzeltony
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Aargh - sorry for causing noise I scrambled commits. Thanks for waking me up :)

@schnitzeltony
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I checked and come to the conclusion it's time for yet another 'sorry for the noise': This patch touches clip names / MidiImportExtender has options for changing track names...

Please sign in to comment.