Skip to content

Commit 9f9341e

Browse files
committed
colorChange indexing should have used curly brackets instead of regular brackets ... redefining colorChange as a cell(100,3) looks even more efficient
1 parent 027666b commit 9f9341e

1 file changed

Lines changed: 8 additions & 8 deletions

File tree

  • src/tools_lgpl/matlab/quickplot/progsrc/private

src/tools_lgpl/matlab/quickplot/progsrc/private/md_print.m

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -425,15 +425,15 @@
425425
ih = get(fig,'inverthardcopy');
426426
set(fig,'inverthardcopy',false)
427427
if isfield(printObj,'InvertHardcopy') && printObj.InvertHardcopy
428-
colorChange = cell(1,100);
428+
colorChange = cell(100,3);
429429
i = 1;
430-
colorChange(i) = {fig,'color',get(fig,'color')};
430+
colorChange(i,:) = {fig,'color',get(fig,'color')};
431431
set(fig,'color','w')
432432
for a = findall(fig,'type','axes')'
433433
for cprop = {'color','xcolor','ycolor','zcolor'}
434434
prop = cprop{1};
435435
i = i+1;
436-
colorChange(i) = {a,prop,get(a,prop)};
436+
colorChange(i,:) = {a,prop,get(a,prop)};
437437
switch prop
438438
case 'color'
439439
set(a,'color','w')
@@ -443,11 +443,11 @@
443443
end
444444
t = get(a,'title');
445445
i = i+1;
446-
colorChange(i) = {t,'color',get(t,'color')};
446+
colorChange(i,:) = {t,'color',get(t,'color')};
447447
end
448-
colorChange = colorChage(1:i);
448+
colorChange = colorChage(1:i,:);
449449
else
450-
colorChange = {};
450+
colorChange = cell(0,3);
451451
end
452452
%
453453
if strcmp(printObj.Name,'Multi page PDF file') && printObj.NextNr > 1
@@ -533,8 +533,8 @@
533533
if ~isempty(normtext)
534534
set(normtext,'fontunits','normalized')
535535
end
536-
for i = 1:length(colorChange)
537-
set(colorChange{i}{:})
536+
for i = 1:size(colorChange,1)
537+
set(colorChange{i,:})
538538
end
539539
set(fig,'inverthardcopy',ih);
540540
%

0 commit comments

Comments
 (0)