luke
/
klayout
mirror of https://github.com/KLayout/klayout.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Fixed #453 (Crash when using 'inverse' feature of TEXT PCell)

This commit is contained in:
Matthias Koefferlein 2019-12-18 18:05:09 +01:00
parent 8fe1223c11
commit a314d3d8c5
2 changed files with 17 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
Changelog
View File

@ -1,5 +1,8 @@
0.26.2 (2019-12-17): 0.26.2 (2019-12-17):
* Bugfix: %GITHUB%/issues/453
A crash happened when the "inverse" button is used on the TEXT
PCell
* Bugfix: %GITHUB%/issues/449 * Bugfix: %GITHUB%/issues/449
The Qt windows styling plugin is now installed too on Windows. The Qt windows styling plugin is now installed too on Windows.
* Bugfix: %GITHUB%/issues/448 * Bugfix: %GITHUB%/issues/448

14
src/edt/edt/edtPCellParametersPage.cc
View File

@ -48,7 +48,9 @@ static void set_value (const db::PCellParameterDeclaration &p, const db::Layout
{ {
QLineEdit *le = dynamic_cast<QLineEdit *> (widget); QLineEdit *le = dynamic_cast<QLineEdit *> (widget);
if (le) { if (le) {
le->blockSignals (true);
le->setText (value.cast<int> ().to_qstring ()); le->setText (value.cast<int> ().to_qstring ());
le->blockSignals (false);
} }
} }
break; break;
@ -57,7 +59,9 @@ static void set_value (const db::PCellParameterDeclaration &p, const db::Layout
{ {
QLineEdit *le = dynamic_cast<QLineEdit *> (widget); QLineEdit *le = dynamic_cast<QLineEdit *> (widget);
if (le) { if (le) {
le->blockSignals (true);
le->setText (value.cast<double> ().to_qstring ()); le->setText (value.cast<double> ().to_qstring ());
le->blockSignals (false);
} }
} }
break; break;
@ -66,7 +70,9 @@ static void set_value (const db::PCellParameterDeclaration &p, const db::Layout
{ {
QLineEdit *le = dynamic_cast<QLineEdit *> (widget); QLineEdit *le = dynamic_cast<QLineEdit *> (widget);
if (le) { if (le) {
le->blockSignals (true);
le->setText (value.to_qstring ()); le->setText (value.to_qstring ());
le->blockSignals (false);
} }
} }
break; break;
@ -75,7 +81,9 @@ static void set_value (const db::PCellParameterDeclaration &p, const db::Layout
{ {
QLineEdit *le = dynamic_cast<QLineEdit *> (widget); QLineEdit *le = dynamic_cast<QLineEdit *> (widget);
if (le) { if (le) {
le->blockSignals (true);
le->setText (value.to_qstring ()); le->setText (value.to_qstring ());
le->blockSignals (false);
} }
} }
break; break;
@ -94,7 +102,9 @@ static void set_value (const db::PCellParameterDeclaration &p, const db::Layout
lp.read (ex); lp.read (ex);
} }
ly->blockSignals (true);
ly->set_current_layer (lp); ly->set_current_layer (lp);
ly->blockSignals (false);
} }
} }
@ -104,7 +114,9 @@ static void set_value (const db::PCellParameterDeclaration &p, const db::Layout
{ {
QCheckBox *cbx = dynamic_cast<QCheckBox *> (widget); QCheckBox *cbx = dynamic_cast<QCheckBox *> (widget);
if (cbx) { if (cbx) {
cbx->blockSignals (true);
cbx->setChecked (value.to_bool ()); cbx->setChecked (value.to_bool ());
cbx->blockSignals (false);
} }
} }
break; break;
@ -120,7 +132,9 @@ static void set_value (const db::PCellParameterDeclaration &p, const db::Layout
int i = 0; int i = 0;
for (std::vector<tl::Variant>::const_iterator c = p.get_choices ().begin (); c != p.get_choices ().end (); ++c, ++i) { for (std::vector<tl::Variant>::const_iterator c = p.get_choices ().begin (); c != p.get_choices ().end (); ++c, ++i) {
if (*c == value) { if (*c == value) {
cb->blockSignals (true);
cb->setCurrentIndex (i); cb->setCurrentIndex (i);
cb->blockSignals (false);
} }
} }
} }