[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Varnamproject-discuss] Suggestions file validity
From: |
Navaneeth K N |
Subject: |
Re: [Varnamproject-discuss] Suggestions file validity |
Date: |
Wed, 21 May 2014 15:31:10 +0530 |
User-agent: |
Mozilla/5.0 (Macintosh; Intel Mac OS X 10.7; rv:24.0) Gecko/20100101 Thunderbird/24.5.0 |
-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA512
Hello Kevin,
It is much better. Few suggestions.
- - is_file_valid should be is_file_exists
- - Don't use printf for reporting error. Set the error using
"set_last_error" call. Writing to the console from a library is not
recommended as you don't know what type of application is using this
library.
On 5/20/14 11:43 PM, Kevin Martin wrote:
> Created a is_file_valid() function in utils.c. I have modified the commit
> as suggested. Please check the 'suggestions_file' branch.
>
> https://github.com/lonesword/libvarnam/tree/suggestions_file
>
- --
Cheers,
Navaneeth
-----BEGIN PGP SIGNATURE-----
Version: GnuPG/MacGPG2 v2.0.22 (Darwin)
Comment: GPGTools - https://gpgtools.org
iQEcBAEBCgAGBQJTfHlmAAoJEHFACYSL7h6kkw4H/2J/AJ12bItBpqzQsRoUTvQG
1sQjGc76q0gzM7Nypjly8T+j5kRu/SgyWotK9WO90X2g/KRIeBxSc38gokdE0YAw
WWuQ5bir/Nmw4MId3tdjekd59fLnRhLN/TSxEVlVE/eNbdsdrGleyKDSglzc2otV
vdcWwX0LjiJa/MPWTdB9cKpdVkQDUhoN13dtgFQaee1MNT+gqiqb9/YcpBOvt5pO
jhjCCsFMZG00z5iksW7PDmcun4Jx70SJ6eWWoVjRlUbhbXZBfTQbtX2Wook0ff1w
PmfDhOkpr8yADDchWnv3yS3mYKxfw9nzrfISPc+anAfuCzjyAofL5aa3iDK3i9s=
=uWX0
-----END PGP SIGNATURE-----
- [Varnamproject-discuss] suggestions file validity, Kevin Martin, 2014/05/16
- [Varnamproject-discuss] Suggestions file validity, Kevin Martin, 2014/05/20
- Re: [Varnamproject-discuss] Suggestions file validity,
Navaneeth K N <=
- Re: [Varnamproject-discuss] Suggestions file validity, Navaneeth KN, 2014/05/22
- Re: [Varnamproject-discuss] Suggestions file validity, Kevin Martin, 2014/05/23
- Re: [Varnamproject-discuss] Suggestions file validity, Navaneeth K N, 2014/05/24
- Re: [Varnamproject-discuss] Suggestions file validity, Kevin Martin, 2014/05/24
- Re: [Varnamproject-discuss] Suggestions file validity, Kevin Martin, 2014/05/24
- Re: [Varnamproject-discuss] Suggestions file validity, Kevin Martin, 2014/05/25
- Re: [Varnamproject-discuss] Suggestions file validity, Navaneeth K N, 2014/05/30
- Re: [Varnamproject-discuss] Suggestions file validity, Kevin Martin, 2014/05/30