Re: [PATCH] dis: Consolidate cmd_dis

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Wed 2015-08-05 17:04 -0400, Dave Anderson wrote:
[ ... ]
> OK, I see what you've done, and "Consolidate" makes perfect sense to me now. 
>  
> While you're at it, the crufty "#ifdef OLDWAY" section should be removed as well,
> since it's just a waste of space.

OK.

> Since the -f implementation should be simple, do you have a problem with
> posting a single patchset that does both the consolidation and the new 
> functionality (and thereby would have a single changelog entry)?

That's no problem at all.

-- 
Aaron Tomlin

--
Crash-utility mailing list
Crash-utility@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/crash-utility



[Index of Archives]     [Fedora Development]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]

 

Powered by Linux