Refactor code handling file.current_zval

[Girgias]

Code refactoring while I'm attempting to get a better grasp of this code.

@twose you wouldn't have a test case which would hit the:

if (SPL_HAS_FLAG(intern->flags, SPL_FILE_OBJECT_READ_CSV)
				&& zend_hash_num_elements(Z_ARRVAL(intern->u.file.current_zval)) == 1) {
			ZEND_ASSERT(false && "Can this happen?");

code path do you? As I don't understand what this branch is attempting to do.

[cmb69]

It seems to me there is the invariant !Z_ISUNDEF(intern->u.file.current_zval) → intern->u.file.current_line. If true, that code is dead (and some other code would be unnecessary, too).

[twose]

Sorry, I have never used fgetcsv() via SPL APIs...