ldforge: comparison src/actions.cc

-:46377d340c23
+:b4a990f59a5e
 	Ui::NewPartUI ui;
 	ui.setupUi (dlg);
 	QString authortext = ld_defaultname;
-	if (!ld_defaultuser.isEmpty())
+	if (not ld_defaultuser.isEmpty())
 		authortext.append (format (" [%1]", ld_defaultuser));
 	ui.le_author->setText (authortext);
 	switch (ld_defaultlicense)
 			QMessageBox::warning (null, "Warning",
 				format ("Unknown ld_defaultlicense value %1!", ld_defaultlicense));
 			break;
 	}
-	if (dlg->exec() == false)
+	if (dlg->exec() == QDialog::Rejected)
 		return;
 	newFile();
 	const LDBFC::Statement BFCType =
 // =============================================================================
 //
 DEFINE_ACTION (Close, CTRL (W))
 {
-	if (!getCurrentDocument()->isSafeToClose())
+	if (not getCurrentDocument()->isSafeToClose())
 		return;
 	delete getCurrentDocument();
 }
 // =============================================================================
 //
 DEFINE_ACTION (CloseAll, 0)
 {
-	if (!safeToCloseAll())
+	if (not safeToCloseAll())
 		return;
 	closeAll();
 }
 DEFINE_ACTION (InsertFrom, 0)
 {
 	QString fname = QFileDialog::getOpenFileName();
 	int idx = getInsertionPoint();
-	if (!fname.length())
+	if (not fname.length())
 		return;
 	QFile f (fname);
-	if (!f.open (QIODevice::ReadOnly))
+	if (not f.open (QIODevice::ReadOnly))
 	{
 		critical (format ("Couldn't open %1 (%2)", fname, f.errorString()));
 		return;
 	}
 	if (fname.length() == 0)
 		return;
 	QFile file (fname);
-	if (!file.open (QIODevice::WriteOnly | QIODevice::Text))
+	if (not file.open (QIODevice::WriteOnly | QIODevice::Text))
 	{
 		critical (format ("Unable to open %1 for writing (%2)", fname, file.errorString()));
 		return;
 	}
 	QDialogButtonBox* const bbx_buttons = new QDialogButtonBox (QDialogButtonBox::Ok | QDialogButtonBox::Cancel);
 	layout->addWidget (te_edit);
 	layout->addWidget (bbx_buttons);
 	dlg->setLayout (layout);
-	dlg->setWindowTitle (APPNAME ": Insert Raw");
+	dlg->setWindowTitle (APPNAME " - Insert Raw");
 	dlg->connect (bbx_buttons, SIGNAL (accepted()), dlg, SLOT (accept()));
 	dlg->connect (bbx_buttons, SIGNAL (rejected()), dlg, SLOT (reject()));
-	if (dlg->exec() == false)
+	if (dlg->exec() == QDialog::Rejected)
 		return;
 	getCurrentDocument()->clearSelection();
 	for (QString line : QString (te_edit->toPlainText()).split ("\n"))
 	QString defaultname = (root.length() > 0) ? format ("%1.png", root) : "";
 	QString fname = QFileDialog::getSaveFileName (g_win, "Save Screencap", defaultname,
 				"PNG images (*.png);;JPG images (*.jpg);;BMP images (*.bmp);;All Files (*.*)");
-	if (fname.length() > 0 && !img.save (fname))
+	if (not fname.isEmpty() && not img.save (fname))
 		critical (format ("Couldn't open %1 for writing to save screencap: %2", fname, strerror (errno)));
 	delete[] imgdata;
 }
 // =============================================================================
 //
 extern_cfg (Bool, gl_axes);
 DEFINE_ACTION (Axes, 0)
 {
-	gl_axes = !gl_axes;
+	gl_axes = not gl_axes;
 	updateActions();
 	R()->update();
 }
 // =============================================================================
 //
 DEFINE_ACTION (VisibilityToggle, 0)
 {
 	for (LDObject* obj : selection())
-		obj->setHidden (!obj->isHidden());
+		obj->setHidden (not obj->isHidden());
 	refresh();
 }
 // =============================================================================
 // =============================================================================
 //
 DEFINE_ACTION (Wireframe, 0)
 {
-	gl_wireframe = !gl_wireframe;
+	gl_wireframe = not gl_wireframe;
 	R()->refresh();
 }
 // =============================================================================
 //
 DEFINE_ACTION (SetOverlay,  0)
 {
 	OverlayDialog dlg;
-	if (!dlg.exec())
+	if (not dlg.exec())
 		return;
 	R()->setupOverlay ((GL::EFixedCamera) dlg.camera(), dlg.fpath(), dlg.ofsx(),
 		dlg.ofsy(), dlg.lwidth(), dlg.lheight());
 }
 // =============================================================================
 //
 DEFINE_ACTION (DrawAngles, 0)
 {
-	gl_drawangles = !gl_drawangles;
+	gl_drawangles = not gl_drawangles;
 	R()->refresh();
 }
 // =============================================================================
 //
 DEFINE_ACTION (testpic, "Test picture", "", "", (0))
 {
 	LDDocument* file = getFile ("axle.dat");
 	setlocale (LC_ALL, "C");
-	if (!file)
+	if (not file)
 	{
 		critical ("couldn't load axle.dat");
 		return;
 	}
 // =============================================================================
 //
 DEFINE_ACTION (BFCView, SHIFT (B))
 {
-	gl_colorbfc = !gl_colorbfc;
+	gl_colorbfc = not gl_colorbfc;
 	updateActions();
 	R()->refresh();
 }
 // =============================================================================
 		defval = selection()[0]->lineNumber();
 	int idx = QInputDialog::getInt (null, "Go to line", "Go to line:", defval,
 		1, getCurrentDocument()->getObjectCount(), 1, &ok);
-	if (!ok || (obj = getCurrentDocument()->getObject (idx - 1)) == null)
+	if (not ok || (obj = getCurrentDocument()->getObject (idx - 1)) == null)
 		return;
 	getCurrentDocument()->clearSelection();
 	obj->select();
 	updateSelection();
 			QDir().mkpath (subdirname);
 		}
 	}
 	// Determine the body of the name of the subfile
-	if (!parentpath.isEmpty())
+	if (not parentpath.isEmpty())
 	{
 		if (parentpath.endsWith (".dat"))
 			parentpath.chop (4);
 		// Remove the s?? suffix if it's there, otherwise we'll get filenames
 	// be carried over to the subfile.
 	for (LDObject* obj : getCurrentDocument()->objects())
 	{
 		LDBFC* bfc = dynamic_cast<LDBFC*> (obj);
-		if (!bfc)
+		if (not bfc)
 			continue;
 		LDBFC::Statement a = bfc->statement();
 		if (a == LDBFC::CertifyCCW || a == LDBFC::CertifyCW || a == LDBFC::NoCertify)

Mercurial > ldforge / file comparison

comparison: src/actions.cc

src/actions.cc