More error leaks
From: Julian Foad <julianfoad_at_btopenworld.com>
Date: 2006-01-19 17:10:21 CET
By putting "__attribute__((warn_unused_result))" on each declaration of a
subversion/libsvn_fs_fs/fs_fs.c: In function 'svn_fs_fs__set_entry':
-- subversion/libsvn_repos/commit.c: In function 'check_authz': subversion/libsvn_repos/commit.c:148: return value of function subversion/libsvn_repos/commit.c: In function 'svn_repos_get_commit_editor4': subversion/libsvn_repos/commit.c:798: return value of function -- subversion/libsvn_repos/dump.c: In function 'dump_node': subversion/libsvn_repos/dump.c:414: return value of 'svn_stream_printf' -- subversion/libsvn_client/locking_commands.c: In function 'svn_client_lock': subversion/libsvn_client/locking_commands.c:457: return value of 'svn_wc_adm_close' subversion/libsvn_client/locking_commands.c: In function 'svn_client_unlock': subversion/libsvn_client/locking_commands.c:514: return value of 'svn_wc_adm_close' -- subversion/libsvn_diff/diff_file.c: In function 'svn_diff__file_output_unified_default_hdr': subversion/libsvn_diff/diff_file.c:894: return value of 'svn_io_stat' -- subversion/svn/main.c: In function 'main': subversion/svn/main.c:825: return value of 'svn_cl__help' subversion/svn/main.c:844: return value of 'svn_cl__help' subversion/svn/main.c:1147: return value of 'svn_cl__help' subversion/svn/main.c:1166: return value of 'svn_cl__help' -- subversion/svn/util.c: In function 'svn_cl__get_log_message': subversion/svn/util.c:631: return value of 'svn_cmdline_prompt_user' -- subversion/svnadmin/main.c: In function 'main': subversion/svnadmin/main.c:1171: return value of 'subcommand_help' subversion/svnadmin/main.c:1196: return value of 'subcommand_help' subversion/svnadmin/main.c:1294: return value of 'subcommand_help' subversion/svnadmin/main.c:1317: return value of 'subcommand_help' subversion/svnadmin/main.c:1335: return value of 'subcommand_help' -- subversion/svndumpfilter/main.c: In function 'main': subversion/svndumpfilter/main.c:1114: return value of 'subcommand_help' subversion/svndumpfilter/main.c:1137: return value of 'subcommand_help' subversion/svndumpfilter/main.c:1168: return value of 'subcommand_help' subversion/svndumpfilter/main.c:1190: return value of 'subcommand_help' subversion/svndumpfilter/main.c:1209: return value of 'subcommand_help' -- subversion/svnlook/main.c: In function 'main': subversion/svnlook/main.c:2020: return value of 'subcommand_help' subversion/svnlook/main.c:2042: return value of 'subcommand_help' subversion/svnlook/main.c:2112: return value of 'subcommand_help' subversion/svnlook/main.c:2142: return value of 'subcommand_help' subversion/svnlook/main.c:2161: return value of 'subcommand_help' subversion/svnlook/main.c:2193: return value of 'subcommand_help' -- subversion/svnsync/main.c: In function 'main': subversion/svnsync/main.c:1110: return value of 'help' subversion/svnsync/main.c:1128: return value of 'help' subversion/svnsync/main.c:1173: return value of 'help' subversion/svnsync/main.c:1187: return value of 'help' subversion/svnsync/main.c:1198: return value of 'help' -- subversion/tests/libsvn_repos/repos-test.c: In function 'commit_editor_authz': subversion/tests/libsvn_repos/repos-test.c:1564: return value of function Most of them probably just need "SVN_ERR", but some may be more complex. If someone is willing to take a look at them and fix all or most of them, that would be great. Of course the other question raised is: do we want to consider using such attributes as a matter of course? Obviously only if it is done neatly and in a way that doesn't interfere with other compilers; macros can help to achieve that. I'll have a bit more of a think about that. - Julian --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org For additional commands, e-mail: dev-help@subversion.tigris.orgReceived on Thu Jan 19 21:27:38 2006 |
This is an archived mail posted to the Subversion Dev mailing list.
This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.