Fix quit confirmation during debuggin session.

[vanshjangir]

Your checklist for this pull request

• I've read the guidelines for contributing to this repository
• I made sure to follow the project's coding style
• I've documented or updated the documentation of every function and struct this PR changes. If not so I've explained why.
• I've added tests that prove my fix is effective or that my feature works (if possible)
• I've updated the rizin book with the relevant information (if needed)

Detailed description

In librz/main/rizin.c , when in debugger session the program ask for quit confirmation independent of scr.confirmquit true or false.
So the quit confirmation when in debug session is removed, and the code for debug session is pasted below the general quit confirm message, Now if user wants to quit then only program will ask to kill the process otherwise the program will just continue

Test plan

...

Closing issues

Closes #3818

[ret2libc]

What if we move the if (debug) code below the

			if (rz_config_get_i(r->config, "scr.confirmquit")) {
				if (!rz_cons_yesno('n', "Do you want to quit? (Y/n)")) {
					continue;
				}
			}

?

It would need some adjustments, but I think it would make more sense. There's some duplicated code here.

[ret2libc]

this is not needed anymore?!

[vanshjangir]

I understood, here is my approach

What I have done here is
if scr.confirmquit is true, then the program will ask for quit confirmation and continue if n is returned.
if y is returned then the program will ask for to kill the process is and will not continue to the loop in any case.

[8dcc]

(Closes #3818)

[ret2libc]

IMO the PR is in good shape. @wargio ?

[wargio]

Thank you <3