[PATCH] Fix for ticket #844

[Claudiu Olteanu]

Ok, I removed that line too. I thought that is more readable in that way,
but you are right: it is better to keep all the declarations in the same
block.

Thanks again for your time,
Claudiu

On Sat, Mar 21, 2015 at 4:50 PM, Lubomir I. Ivanov <neolit123 at gmail.com>
wrote:

> On 21 March 2015 at 16:44, Claudiu Olteanu
> <olteanu.vasilica.claudiu at gmail.com> wrote:
> > Oh, thanks. I usually insert more tabs and finally align with a number
> > of spaces less than the size of a tab.
> >
> > Hope that the last patch (attached to this e-mail) is good :-).
> >
>
> yeah, the patch is good now.
> line 31 adds a line that is not needed, but a line separating the
> variable declaration and the function body is fine...
>
> we just try to keep the whitespace of the source code very sanitized.
> just browse the file tree and you can see the generics.
> also refer to CodingStyle and the Linux Kernel coding style.
>
> thanks
> lubomir
> --
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.subsurface-divelog.org/pipermail/subsurface/attachments/20150321/0a18721d/attachment.html>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-Implement-handler-for-cancel-button-of-Save-As-dialo.patch
Type: text/x-patch
Size: 1590 bytes
Desc: not available
URL: <http://lists.subsurface-divelog.org/pipermail/subsurface/attachments/20150321/0a18721d/attachment.bin>