Details
-
Type:
Bug
-
Status:
Resolved
-
Priority:
Critical
-
Resolution: Cannot Reproduce
-
Affects Version/s: None
-
Fix Version/s: Sprint 2011-32
-
Component/s: iOS
-
Environment:
- IOS 4.2, IOS 4.3
- Titanium 1.7.0, 1.7.1
- mac os 1.6.7
Description
If you use setdata on IOS the tableviewrow dont release. If I compile with 1.7.0 Release Candidate there is no problems. This only appears if I try to compile to 1.7.0 final or 1.7.1. Using Intruments its clear the behavior difference between 1.7 RC and 1.7.1 and 1.7.0
Test case bellow:
var win = Titanium.UI.createWindow(
{
title: 'Tableview Test'
});
var runNumber = 0;
var tableview = Titanium.UI.createTableView({
backgroundColor: 'white',
border: 1,
borderColor: '#E0E0E0',
width: 315,
height: 450,
left: 35,
top: 75,
allowsSelection: true,
separatorStyle: Ti.UI.iPhone.TableViewSeparatorStyle.SINGLE_LINE
});
win.add(tableview);
var buttomSetData = Titanium.UI.createButton({
title: 'Set data',
bottom: 50,
right: 480,
border: 1,
height: 100,
width: 47
});
buttomSetData.addEventListener('click', function (e) {
var tableviewData =[];
for(var i=0;i<200;i++)
{
var row1 = Ti.UI.createTableViewRow();
var infoLabel = Titanium.UI.createView({
height: 12,
width: 12,
//backgroundImage :'images/delrow.png',
backgroundColor: 'red',
top: 8,
left: 20
});
var textLabel = Ti.UI.createLabel({
color: '#000',
text: 'Run number ' + runNumber + ' Row number ' + i,
font: {
fontSize: 15
},
top: 5,
left: 50,
height: 20,
width: 260
});
row1.add(textLabel);
row1.add(infoLabel);
tableviewData.push(row1);
}
tableview.setData(tableviewData);
runNumber++
});
win.add(buttomSetData);
win.open();
Activity
Hide
Tim Poulsen
added a comment -
I can reproduce this problem with both 1.7.2 final and 1.8.0.v20110915133349. Watching via instruments shows row proxies are not being released or reused; memory grows continuously as you "reset" the table.
function setRows(_start) {
var rows = [];
var end = _start + 100;
for(var i = _start; i < end; i++) {
var row = Ti.UI.createTableViewRow({
title:'Test '+i
});
rows.push(row);
}
return rows;
}
var table = Ti.UI.createTableView({
data: setRows([]) ,
bottom: 70
});
var button = Ti.UI.createButton({
title: "Reset Data",
height: 50,
width: 200,
bottom: 10
});
var click = 0;
button.addEventListener("click", function() {
click += 100;
table.setData(setRows(click));
// The following doesn't release memory/row proxies either
// table.data = setRows(click);
// nor does
// table.data = [];
// table.setData(setRows(click));
});
win1.add(table);
win1.add(button);
See also http://jira.appcelerator.org/browse/TIMOB-3799 and http://developer.appcelerator.com/question/120923/tableview-setdata-doesnt-clear-memory
Show
Tim Poulsen
added a comment - I can reproduce this problem with both 1.7.2 final and 1.8.0.v20110915133349. Watching via instruments shows row proxies are not being released or reused; memory grows continuously as you "reset" the table.
function setRows(_start) {
var rows = [];
var end = _start + 100;
for(var i = _start; i < end; i++) {
var row = Ti.UI.createTableViewRow({
title:'Test '+i
});
rows.push(row);
}
return rows;
}
var table = Ti.UI.createTableView({
data: setRows([]) ,
bottom: 70
});
var button = Ti.UI.createButton({
title: "Reset Data",
height: 50,
width: 200,
bottom: 10
});
var click = 0;
button.addEventListener("click", function() {
click += 100;
table.setData(setRows(click));
// The following doesn't release memory/row proxies either
// table.data = setRows(click);
// nor does
// table.data = [];
// table.setData(setRows(click));
});
win1.add(table);
win1.add(button);
See also http://jira.appcelerator.org/browse/TIMOB-3799 and http://developer.appcelerator.com/question/120923/tableview-setdata-doesnt-clear-memory
Thanks for raising this ticket. Please remember to state the build date and hash of the Titanium SDK you have tested. You will find this information in the log on application launch. Please read the Jira Ticket Checklist for future tickets.
Cheers