Search results

  1. MCroft

    Bug - Waiting for Info 1952 Mickey mantle pick in Deck of Every Card crash

    would it help to move from Synchronized to java.util.concurrent.locks.ReadWriteLock? They'd still be locked against writes, but they'd allow multiple reads.
  2. MCroft

    Bug - Waiting for Info 1952 Mickey mantle pick in Deck of Every Card crash

    That's been a hard one to build good tests for. We worked mostly with Preference.increment(), because it exercised read and write and depended on the before-state. But perhaps we need to move on from synchronized() to tryLock() , or better ReadLock()/WriteLock(). (or other lock methods if...
  3. MCroft

    OLVED, use alias] Custom daily deed not working - issue with commas that get piped

    Hmm. I like either running a script or an alias, but would it be useful to have escapes in there?
  4. MCroft

    Bug Maximizer Bug

    would also recommend adding the dump parameter to the maximizer string to get more information about what is being considered.
  5. MCroft

    OLVED, use alias] Custom daily deed not working - issue with commas that get piped

    The docs are here: https://wiki.kolmafia.us/index.php/Daily_Deeds You might want to see if this advice applies. It's for the $TEXT type, but it might solve your problem. If you want to use a comma in your text, immediately follow the comma with a pipe character so it will not be parsed as...
  6. MCroft

    Bug - Not A Bug Chat message deletion problem

    It's not a bad feature request, so feel free to put it in as a new one.
  7. MCroft

    Bug - Not A Bug Chat message deletion problem

    Pressing the up or down arrow populates the text entry with the prior message or the next one if there is one (after pressing the up arrow, for instance. In the gCLI, the same phenomena occurs. When you overwrite the text entry, the list of prior messages does not add the current string to...
  8. MCroft

    Bug - Fixed Missing relay script dropdown

    I did some spot testing and couldn't make it reproduce. I tried a lot of things, including installing and removing scripts while I had that open. the @ckb menu worked really badly if it was on icon menu and we loaded his stuff, but it never told me it was detecting incorrectly or glitched. I...
  9. MCroft

    Bug - Fixed Missing relay script dropdown

    many fewer messages. https://www.kingdomofloathing.com/awesomemenu.php?pwd&action=toggle From around line 781
  10. MCroft

    Bug - Fixed Missing relay script dropdown

    Oh, the version on mine is because I'm working from my IDE. That version number is off, but I was testing with your development branch before the PR was approved. Otherwise, I'm working in Preferences.java and PreferencesTest.java, so it shouldn't have any affect on this testing, but if you...
  11. MCroft

    Bug - Fixed Missing relay script dropdown

    I was also looking for events attached to the elements there, but I think some of it is round-tripping to the PHP server. I'll grab debug as well. Would trace be of any help?
  12. MCroft

    Bug - Fixed Missing relay script dropdown

    So, I tried the new version. I couldn't break it, but I did see the error message. The fix seems to be working so far for me for this test case (I don't think this is the only path to the problem, though). A few times KoLmafia detected NORMAL when it was FANCY and KoLmafia thought it was...
  13. MCroft

    r26152 - BasicScope obsolete comments by @fredg1 in

    Well, I have no friends who are still playing, so I don't get gifts. I am probably rare in that respect. Most people give up before I do. I typically view the descriptions in the relay browser by clicking on the icon and viewing the documentation in the browser window it opens. I think that...
  14. MCroft

    r26152 - BasicScope obsolete comments by @fredg1 in

    Maybe it's not, or not for this discussion. I was trying to suggest that my way of playing wasn't the only way of playing and people who do use the mini-browser might not be as surprised by the method of displaying the message. I also always want the funny/cute messages. For me, the gCLI...
  15. MCroft

    r26152 - BasicScope obsolete comments by @fredg1 in

    Maybe we could send it to the gCLI. The mini browser isn’t something that’s normally part of my play style.
  16. MCroft

    r26152 - BasicScope obsolete comments by @fredg1 in

    PR for rollback. I asked you to review. https://github.com/kolmafia/kolmafia/pull/1855
  17. MCroft

    r26152 - BasicScope obsolete comments by @fredg1 in

    @Veracity I added a preference and didn't make it a default so that it didn't impact anyone who did want the behavior you wanted. You asked if there was a bug report. That was one. This is another: https://kolmafia.us/threads/use-seal-tooth-opens-window.25830/#post-161201 I didn't understand...
  18. MCroft

    r26152 - BasicScope obsolete comments by @fredg1 in

    the motivation was the swing GUI popup on using a Seal Tooth, which had been reported a few times. The example below is just one such. https://kolmafia.us/threads/gcli-command-use-seal-tooth-always-opens-a-documentation-window-unexpectedly.25625/ If we want to revisit it (or if you want me to...
  19. MCroft

    r26152 - BasicScope obsolete comments by @fredg1 in

    A while back, but it's one of mine. Happy to discuss and revise, but I'll have to rehydrate my brain first...
  20. MCroft

    Daily storage budget

    The budget is used to tell KoLmafia how many pulls it can use to fulfill requests to do stuff. I dont have a comprehensive mental map of all the places it comes into play, but your case is similar to the way I use it: to allow me to build/pull awesome drinks in the GUI instead of relying on...
Back
Top