Openwall GNU/*/Linux - a small security-enhanced Linux distro for servers
[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Date: Mon, 24 Jun 2013 00:04:59 +0530
From: Sayantan Datta <std2048@...il.com>
To: john-dev@...ts.openwall.com
Subject: Re: Sayantan's weekly report #0

On Wednesday 12 June 2013 09:55 PM, Sayantan Datta wrote:
> On Tuesday 11 June 2013 01:14 PM, Solar Designer wrote:
>> Unrelated to your priorities listed above, how about one more: improve
>> your code quality?  Have you done anything on this, such as in response
>> to me and Lukas (IIRC) pointing out some issues to you recently?
> I did the code clean up few days back when you mentioned about those 
> issues in my code. Please check the bleeding.  I have removed all 
> redundant opencl host code, corrected most of the weired alignment and 
> removed false blank lines which weren't required. Lukas said to do
>
> indent -kr -i8 -nlp -nbbo -ncs -l79 -lc79 <filename>
>
> But this breaks down long lines into multiple shorter lines at wrong 
> places that makes the code unreadable from my perspective. So I didn't 
> do that.
>
> You also mentioned about weired naming , while I have fixed some of 
> them but not all because I can't think of any better names which 
> explains the identifiers better. Do you prefer hungarian notation ?
>
> Regards,
> Sayantan
>
>
>
The cleaned codes are only in bleeding branch of magnum's git repo.

Regards,
Sayantan

Powered by blists - more mailing lists

Your e-mail address:

Powered by Openwall GNU/*/Linux - Powered by OpenVZ