Short-circuit return on zero input in cyl_bessel_y() #71
  Add this suggestion to a batch that can be applied as a single commit.
  This suggestion is invalid because no changes were made to the code.
  Suggestions cannot be applied while the pull request is closed.
  Suggestions cannot be applied while viewing a subset of changes.
  Only one suggestion per line can be applied in a batch.
  Add this suggestion to a batch that can be applied as a single commit.
  Applying suggestions on deleted lines is not supported.
  You must change the existing code in this line in order to create a valid suggestion.
  Outdated suggestions cannot be applied.
  This suggestion has been applied or marked resolved.
  Suggestions cannot be applied from pending reviews.
  Suggestions cannot be applied on multi-line comments.
  Suggestions cannot be applied while the pull request is queued to merge.
  Suggestion cannot be applied right now. Please check back later.
  
    
  
    
Over in #69, there were about 30 failures of the
cyl_bessel_y()tests in the Windows CI job. This lineappears 30 times in the log, and in each case the
zinput is 0. Apparently the culprit is somewhere in here:xsf/include/xsf/bessel.h
Lines 963 to 970 in 33768a0
In this PR, I avoid that code by handling
z=0+0jentirely in the first if-statemet where it checks forz = 0+0j. With this change, I don't see any of thosecyl_bessel_y()test failures (but there are many others, so the Windows job still fails overall).This change is not thoroughly checked, and is probably not the most elegant way to implement the calculation. I'm sure I'm duplicating functionality that should be in that code shown above, so I'm leaving this in draft status. I think this gets close to the correct behavior, so it might be useful for further investigation and debugging. Ideally the specific problem in the above code that occurs on Windows would be identified and fixed, and this PR closed without merging.