Refactor omnisharp-find-implementations a bit
omnisharp--write-lines-to-compilation-buffer: document lines-to-write
Add omnisharp--write-quickfixes-to-compilation-buffer function
Whitespace fixes
Make implementations and usages code cleaner
Add omnisharp--find-file-possibly-in-other-window
Fix omnisharp--buffer-exists-for-file-name working only for cur file
omnisharp--find-file-possibly-in-other-window style improvements

Do not allow filename to be nil, ever.
omnisharp-go-to-file-line-and-column-worker: fix logical error
Add omnisharp--get-common-params-for-emacs-side-use

This is the ugliest function ever. But adding it makes renaming work
so much simpler... ;__;
omnisharp--buffer-exists-for-file-name: optimise a bit
omnisharp-go-to-file-line-and-column: update documentation
find-file-possibly-in-other-window: use a buffer if one exists.

This change avoids setting point to the beginning of the file after
each call of this function. Now the position of point is preserved if
a buffer exists for the given filename.
find-file-possibly-in-other-window: finish comments
omnisharp--set-buffer-contents-to: make line and column optional
Fix rename messing with current point position in buffers.

Also refactor it a bit. Now the worker simply calls the API and
returns its result.