On Thu, Sep 29, 2011 at 02:58:23PM +0100, Måns Rullgård wrote: > Diego Biurrun <[email protected]> writes: > > > On Thu, Sep 29, 2011 at 12:07:41PM +0100, Måns Rullgård wrote: > >> Diego Biurrun <[email protected]> writes: > >> > On Thu, Sep 29, 2011 at 11:46:26AM +0100, Måns Rullgård wrote: > >> >> Kostya Shishkov <[email protected]> writes: > >> >> > On Wed, Sep 28, 2011 at 09:52:26PM +0100, Mans Rullgard wrote: > >> >> >> + int8x8_fmul_int32(subband_samples[k][l], > >> >> >> + > >> >> >> &high_freq_vq[s->high_freq_vq[k][l]][subsubframe * 8], > >> >> >> + s->scale_factor[k][l][0]); > >> >> > > >> >> > some Diego might complain about identation here > >> >> > >> >> This looked more sane than all other alternatives. > >> > > >> > more sane than > >> > > >> > int8x8_fmul_int32(subband_samples[k][l], > >> > &high_freq_vq[s->high_freq_vq[k][l]][subsubframe * > >> > 8], > >> > s->scale_factor[k][l][0]); > >> > >> Look at the context. At the indentation level of the block it's in, > >> that would push the longer line over 80 columns. > > > > I can live with that, better in fact, than with the non-standard and > > arguably messed-up indentation. > > That's because you're not the one reading the code.
Usually the most frequent reader doesn't mind any formatting, being compiler and such. _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
