detect/msg: convert to FAIL/PASS API

[jufajardini]

Issue 4053. Adjust code formatting style (wrap long lines).
Replace SigInit with DetectEngineAppendSig.

Make sure these boxes are signed before submitting your Pull Request -- thank you.

• I have read the contributing guide lines at https://redmine.openinfosecfoundation.org/projects/suricata/wiki/Contributing
• I have signed the Open Information Security Foundation contribution agreement at https://suricata-ids.org/about/contribution-agreement/
• I have updated the user guide (in doc/userguide/) to reflect the changes made (if applicable)

Link to redmine ticket:
https://redmine.openinfosecfoundation.org/issues/4053

Link to previous PR:
#5502

Describe changes:

• Wrap long lines to keep code complaint with formatting guidelines
• Replace SigInit with DetectEngineAppendSig
• Remove no longer necessary SigFree

PRScript output (if applicable):

#suricata-verify-pr:
#suricata-verify-repo:
#suricata-verify-branch:
#suricata-update-pr:
#suricata-update-repo:
#suricata-update-branch:
#libhtp-pr:
#libhtp-repo:
#libhtp-branch:

[jufajardini]

I've run clang and already detected what happened. Should have run it once again before submitting... Sorry for not doing that. Lesson learned for the next time...

In any case, I'll wait to check if there's anything else that I should change...

[victorjulien]

Looks good to me. If you do a new PR with the clang-format fixes included (same commit) then I think we're good.

[jufajardini]

Replaced by #5506