> But no, it does not fix the "excessive size" limit in the email updates<br>
<br>
Then this might be another to-do for SqueakSource at any point in time: Increase this limit? I don't think we lack the necessary resources today. Maybe we should also support something like "diff excerpts" without printing out all unchanged lines. However, nitpicking only. :-)<br>
<br>
Best,<br>
Christoph<br>
<br>
<font color="#808080">---<br>
</font><font color="#808080"><i>Sent from </i></font><font color="#808080"><i><a href="https://github.com/hpi-swa-lab/squeak-inbox-talk"><u><font color="#808080">Squeak Inbox Talk</font></u></a></i></font><br>
<br>
On 2022-02-24T19:05:54-05:00, lewis@mail.msen.com wrote:<br>
<br>
> On Thu, Feb 24, 2022 at 09:44:48PM +0000, Thiede, Christoph wrote:<br>
> > +1, maybe this change will even eliminate the "(excessive size, no diff calculated)" messages on the list? :-)<br>
> > <br>
> > <br>
> > Best,<br>
> > <br>
> > Christoph<br>
> > <br>
> <br>
> Yes it makes the diffing in the image readable. But no, it does not fix<br>
> the "excessive size" limit in the email updates, see the commit notice at<br>
> http://lists.squeakfoundation.org/pipermail/squeak-dev/2022-February/219046.html<br>
> for example.<br>
> <br>
> For that I guess we would need to generate a context diff. But I don't<br>
> think that this is very important, the main thing is that you can now<br>
> see the method changes in the image.<br>
> <br>
> Dave<br>
> <br>
> <br>
> > ________________________________<br>
> > Von: Squeak-dev <squeak-dev-bounces at lists.squeakfoundation.org> im Auftrag von Taeumel, Marcel<br>
> > Gesendet: Mittwoch, 9. Februar 2022 17:57:49<br>
> > An: squeak-dev<br>
> > Betreff: Re: [squeak-dev] The Inbox: System-dtl.1304.mcz<br>
> > <br>
> > Hi Dave --<br>
> > <br>
> > Nice idea. Well, #privateAuthorsRaw was meant to be a small database. Like those base64 encoded icon contents. If you really want to see nicer diffs for this method, go ahead at update it. :-) Then we can also see whether we remove something by accident.<br>
> > <br>
> > Best,<br>
> > Marcel<br>
> > <br>
> > Am 09.02.2022 02:54:29 schrieb commits at source.squeak.org <commits at source.squeak.org>:<br>
> > <br>
> > A new version of System was added to project The Inbox:<br>
> > http://source.squeak.org/inbox/System-dtl.1304.mcz<br>
> > <br>
> > ==================== Summary ====================<br>
> > <br>
> > Name: System-dtl.1304<br>
> > Author: dtl<br>
> > Time: 8 February 2022, 8:54:15.36808 pm<br>
> > UUID: 425a12bd-d324-4d48-91db-a29dfb4bb67b<br>
> > Ancestors: System-mt.1303<br>
> > <br>
> > Use Character cr as a separator in privateAuthorsRaw so that each author appears on a separate line in the method. Future updates will be readable in the version diffs.<br>
> > <br>
> > =============== Diff against System-mt.1303 ===============<br>
> > <br>
> > Item was changed:<br>
> > ----- Method: SystemNavigation class>>initializeAuthors (in category 'class initialization') -----<br>
> > initializeAuthors<br>
> > "self initializeAuthors"<br>
> > <br>
> > | s currentName currentAbbrv |<br>
> > Authors := Dictionary new.<br>
> > s := self privateAuthorsRaw readStream.<br>
> > <br>
> > [s atEnd] whileFalse: [<br>
> > "Read author name"<br>
> > currentName := String streamContents: [:as |<br>
> > [s peek ~= $#] whileTrue: [as nextPut: s next]].<br>
> > Authors at: currentName put: Set new.<br>
> > "Read abbreviations"<br>
> > s next. "skip $#"<br>
> > currentAbbrv := ''.<br>
> > + [s atEnd not and: [s peek ~= Character cr]] whileTrue: [<br>
> > - [s atEnd not and: [s peek ~= $!!]] whileTrue: [<br>
> > s peek = $# ifTrue: [<br>
> > (Authors at: currentName) add: currentAbbrv.<br>
> > currentAbbrv := ''.<br>
> > s next. "skip $#"].<br>
> > currentAbbrv := currentAbbrv, s next asString. "slow..."<br>
> > ].<br>
> > currentAbbrv ifNotEmpty: [(Authors at: currentName) add: currentAbbrv].<br>
> > s next. "skip $!!"<br>
> > ].<br>
> > <br>
> > "Fill inverted index for authors."<br>
> > AuthorsInverted := Dictionary new.<br>
> > Authors keysAndValuesDo: [:fullName :abbreviations |<br>
> > abbreviations do: [:abbrv |<br>
> > (AuthorsInverted<br>
> > at: abbrv<br>
> > ifAbsentPut: [Set new]) add: fullName]].!<br>
> > <br>
> > Item was changed:<br>
> > ----- Method: SystemNavigation class>>privateAuthorsRaw (in category 'class initialization') -----<br>
> > (excessive size, no diff calculated)<br>
> > <br>
> > <br>
> <br>
> ><br>
> <br>
>