Fix an oversight in the new 'Record trace' checkbox in the trace dialog

README.md

@@ -78,5 +78,6 @@ You can find an exhaustive list of GitHub contributors [here](https://github.com
 - [kobalicek](https://github.com/kobalicek)
 - [athre0z](https://github.com/athre0z)
 - [ZehMatt](https://github.com/ZehMatt)
+- [mrfearless](https://twitter.com/fearless0)
 
 Without the help of many people and other open-source projects, it would not have been possible to make x64dbg what is it today, thank you!

src/dbg/plugin_loader.cpp

@@ -1049,7 +1049,7 @@ bool pluginexprfuncregisterex(int pluginHandle, const char* name, const ValueTyp
 
     std::vector<ValueType> argTypesVec(argCount);
 
-    for(auto i = 0; i < argCount; i++)
+    for(size_t i = 0; i < argCount; i++)
         argTypesVec[i] = argTypes[i];
 
     if(!ExpressionFunctions::Register(name, returnType, argTypesVec, cbFunction, userdata))

src/gui/Src/Gui/SimpleTraceDialog.cpp

@@ -53,7 +53,7 @@ void SimpleTraceDialog::on_btnOk_clicked()
         if(msgyn.exec() == QMessageBox::No)
             return;
     }
-    if(ui->chkRecordTrace->isChecked())
+    if(ui->chkRecordTrace->isChecked() && !TraceBrowser::isRecording())
     {
         if(!TraceBrowser::toggleTraceRecording(this))
         {