You're right Phill, the non-printing bytes do specify the length of the following field/label. A rewrite based on this info should be more efficient.
This may also avoid the need for stripping ascii control characters. On May 1, 2012 1:13 AM, "phill" <[email protected]> wrote: > One more thing, lines 223 - 229: the "non-printing byte" specifies how many > chars (or bytes) long the "label" is. It is not unique to the label, i.e. > If you had two labels the same length, the would have the same non-printing > byte ay the start. > > Phill > > -- > > https://code.launchpad.net/~sfindlay/openlp/songs-import-powersong/+merge/104102 > You are the owner of lp:~sfindlay/openlp/songs-import-powersong. > -- https://code.launchpad.net/~sfindlay/openlp/songs-import-powersong/+merge/104102 Your team OpenLP Core is requested to review the proposed merge of lp:~sfindlay/openlp/songs-import-powersong into lp:openlp. _______________________________________________ Mailing list: https://launchpad.net/~openlp-core Post to : [email protected] Unsubscribe : https://launchpad.net/~openlp-core More help : https://help.launchpad.net/ListHelp

