aboutsummaryrefslogtreecommitdiffstats
path: root/Models
diff options
context:
space:
mode:
authorMax <post@wickenrode.com>2014-12-26 20:37:39 +0100
committerMax <post@wickenrode.com>2014-12-26 20:37:39 +0100
commitc7362cc8c6d3c02d8e9b5f5a31735bb3fbbf0b02 (patch)
tree3c22d79aa7532f80999d77236ccca0970c44c3ab /Models
parent03d11e5d82a7c2a0c5db624020632a7d10b5a13a (diff)
downloadsequelpro-c7362cc8c6d3c02d8e9b5f5a31735bb3fbbf0b02.tar.gz
sequelpro-c7362cc8c6d3c02d8e9b5f5a31735bb3fbbf0b02.tar.bz2
sequelpro-c7362cc8c6d3c02d8e9b5f5a31735bb3fbbf0b02.zip
************** IMPORTANT: SEE BELOW **************
This change should fix the "slow CSV import dialog" issue on 10.10 (details below). To accomplish that, some major changes had to be made to the CSV import code. PLEASE VERIFY THE CORRECTNESS OF THE NEXT FEW CSV IMPORTS YOU DO! * Renamed some variables for better search visibility * Changed some instances where NSNumbers were being compared using "==" instead of "isEqual…" (does not work with object literals!) * The CSV import dialog would recreate the popup menus on *every* call to "tableView:objectValueFor…". However Instruments suggests that [NSMenu removeAllItems] and [NSMenu addItemsWithTitles:] are **really** expensive, esp. when called multiple times per second (e.g. when scrolling in the table view). This commit moves the popup menu setup into its own method and only calls it when the data actually changes. * The 'destination table column' was for some reason a NSPopupButtonCell which only ever had one item and sometimes got changed to a NSTextFieldCell in code. I didn't see any need to keep the popup cell and replaced it with a simple text cell.
Diffstat (limited to 'Models')
0 files changed, 0 insertions, 0 deletions