iceman1001 / proxmark3

[Deprecated] Iceman Fork, the most totally wicked fork around if you are into proxmark3
http://www.icedev.se/pm3.aspx
GNU General Public License v2.0
463 stars 116 forks source link

CmdFSKsimTAG and CopyHIDtoT55x7 debug messages #251

Closed Defensor7 closed 5 years ago

Defensor7 commented 5 years ago

Added debug messages to let client understand current status.

iceman1001 commented 5 years ago

Drop the first debug message, I just dislike those. And the second one, put it in a debug level if statment,

iceman1001 commented 5 years ago

I have spent time removing some message that doesn't give any debug information at all. The prefered pattern to follow is to make sure when exiting from device function, is to send a ACK cmd, and let the client wait for it and print somthing there. So that first adding is no-go.

The second addition I can accept it you add

 if (MF_DBGLEVEL > 3)