| To: | Adrian Bunk <bunk@xxxxxxxxx> |
|---|---|
| Subject: | [xfs-masters] Re: [2.6 patch] fs/xfs/: possible cleanups |
| From: | Christoph Hellwig <hch@xxxxxxxxxxxxx> |
| Date: | Fri, 13 May 2005 09:44:54 +0100 |
| Cc: | Andrew Morton <akpm@xxxxxxxx>, xfs-masters@xxxxxxxxxxx, nathans@xxxxxxx, linux-xfs@xxxxxxxxxxx, linux-kernel@xxxxxxxxxxxxxxx |
| In-reply-to: | <20050513004721.GS3603@stusta.de> |
| Mail-followup-to: | Christoph Hellwig <hch@xxxxxxxxxxxxx>, Adrian Bunk <bunk@xxxxxxxxx>, Andrew Morton <akpm@xxxxxxxx>, xfs-masters@xxxxxxxxxxx, nathans@xxxxxxx, linux-xfs@xxxxxxxxxxx, linux-kernel@xxxxxxxxxxxxxxx |
| References: | <20050513004721.GS3603@stusta.de> |
| Reply-to: | xfs-masters@xxxxxxxxxxx |
| Sender: | xfs-masters-bounce@xxxxxxxxxxx |
| User-agent: | Mutt/1.4.1i |
On Fri, May 13, 2005 at 02:47:21AM +0200, Adrian Bunk wrote: > This patch contains the following possible cleanups: > - make needlessly global code static > - remove the obsolete support/qsort.c > - debug.c: remove the read-only global variable doass > (replaced by a #define) > - #if 0 the following unused global functions: > - quota/xfs_dquot.c: xfs_qm_dqwarn > - xfs_bmap_btree.c: xfs_bmbt_lookup_le > - xfs_fsops.c: xfs_fs_log_dummy > - xfs_inode.c: xfs_inobp_bwcheck > - xfs_trans.c: xfs_trans_callback > - xfs_trans_inode.c: xfs_trans_ihold_release > > Please check which of these changes do make sense. I'll take care of getting this patch into the XFS tree. Hopefully we can just remove the unused global functions, but I'll have to check first whether they might be used by the userland tools. |
| <Prev in Thread] | Current Thread | [Next in Thread> |
|---|---|---|
| ||
| Previous by Date: | [xfs-masters] It is scary to check the highprices on tablets at the local store., jack carrasquillo |
|---|---|
| Next by Date: | [xfs-masters] 小川美樹&早川あゆみ, お蔭様で大盛況!! |
| Previous by Thread: | [xfs-masters] It is scary to check the highprices on tablets at the local store., jack carrasquillo |
| Next by Thread: | [xfs-masters] 小川美樹&早川あゆみ, お蔭様で大盛況!! |
| Indexes: | [Date] [Thread] [Top] [All Lists] |