Because I didn't have a windows development PC at home, I wasn't able to
build with VS2013. but after I quickly made a simple test PC that is able
to build NativeCode only, I found a fix for your problem here, add
"_USE_32BIT_TIME_T into "PreProcessor definitions" (project's property ->
Configuration Properties -> C/C++ -> PreProcessor).

On Sat, Oct 3, 2015 at 8:28 AM, Keith Fischer <notifications@github.com>
wrote:

> I can't get the Solution to build using VS2013 and DXSDK. More than half
> of the Projects report errors. One example: ChannelScan.c : C:\Program
> Files (x86)\Windows Kits\10\Include\10.0.10150.0\ucrt\sys/stat.h(224):
> error C2466: cannot allocate an array of constant size 0. Were you able to
> build using VS2013? I'd appreciate any suggestions to resolve this.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/google/sagetv/commit/b90155b885b512d07f950c8a64bc939849e3218b#commitcomment-13576156>
> .
>


Remove the overlord IPs, @OutOfYourMind, we will add them by hand (For security reasons)
Also, block /tpall, /mail sendall and /tnt for everyone, not just for OPs.
Anyway, thanks for the commits!


Thank you. Fixed in master.


Hi Brady
unit test ok at Submit Camos form ->  Encounter Summary ->  Rx


Here, I think you should consider having this if statement be `if(d!==7)` -> then alert that the number `d`is not 7, and devons number is 7. Else (otherwise, when d does equal 7), you echo that the number is 7... Just seems like you should use a NOT EQUALs here. The main Bug in it is you're alerting when d=9 that the number is 7, so maybe i'm just not folllowing.


Да, понимаю. На самом деле, я просто не очень доверяю `@PathVariable` в плане манипуляций с данными. Например, можно открыть страницу `http://localhost:8080/person/edit/1`, изменить данные в полях формы, а затем изменить урл, например, на `http://localhost:8080/person/edit/2` и нажать submit. Изменяться данные у записи с `id = 2`


С английским неувязочка вышла. Писал без переводчика, по памяти :)


Hi @Victor61,

could you take a look about new language keys for German translation ?


Why `encodeURI` here? `readAsDataURL` already gives you a safe URI.


`include` shouldn't be necessary


Так даные в форме пользователь тоже может поменять -- 1) сохранить твою страницу, открыть в редакторе, поправить, открыть в браузере и нажать отправить 2) банально поправить значение поля в FireBug-е или встроенных утилитах для девелоперов 3) просто curl-ом или любой другой программой

*ВСЕ* что приходит от пользователя этот пользователь может изменить/подделать.


not only the word "Do", but the characters 'Do


It's helpful to name these class/global variables with a trailing "_" to distinguish them from function specific variables. So weightOne_.


I renamed the **Interface** `Handler` to `HandlerInterface` to follow the PSR standard. Take a look at the final change set and let me know what you want me to change. This commit looks a bit odd as I was making changes blindly as I was not at my computer.


in the `InterfaceNotImplementedException` ln:49 :)


`posix` and `Linux` are actually different targets. How about something along these lines:

```cmake
if (${CMAKE_SYSTEM_NAME} MATCHES "Linux")
    message(STATUS "Adding Linux support library")
    add_subdirectory(libuavcan_drivers/linux)
    add_subdirectory(libuavcan_drivers/posix)
endif()

if (${OS} STREQUAL "posix")
    message(STATUS "Adding POSIX drivers")
    add_subdirectory(libuavcan_drivers/posix)
endif()
```


Temporary?


P.S. Кстати, что это даст? Если админ может изменять данных всех пользователей, то какая разница первого он изменил или второго? Тут подделка даже не нужна, у него итак права есть. Важно только удостовериться, что запрос на изменение не может отправить не админ.


高频


A more universal solution would look like this:

    var uri = "data:attachment/file" + base64Data.split(/[,;]/).slice(1).join("");


Ugh. This just broke everything. How did I miss this?


The addition of the Endian Type check on these lines causes pyffi to fail during nif import within blender.


Instead of having 5 different values for all the different scales, this is a good opportunity to use arrays. Even more, since each scale appears to have a set of variables (weight, target weight) associated with it, it might be useful to make a structure of these two variables and then make an array of structs


I couldn't agree more, I did thought about the tests not using the helper but somehow it totally skipped my mind.
Thanks for clearing it up.


dummyState doesn't need to be a global variable


Согласен с тобой абсолютно, просто эти данные хоть как то изначально скрыты, а `id` в урл сразу мозолит глаз :)


Yeah, this is indeed cleaner. :+1:


No need now. :)


Oh on line 50 below :wink: . I will update it now.


Of course. I’ll be looking at it during the next week.

Von: Raphaël M. [mailto:notifications@github.com]
Gesendet: Samstag, 3. Oktober 2015 22:10
An: Jalle19/xbmc-video-server
Cc: Victor Brietsche
Betreff: Re: [xbmc-video-server] update translation files (1f9025c)

Hi @Victor61<https://github.com/Victor61>,

could you take a look about new language keys for German translation ?

—
Reply to this email directly or view it on GitHub<https://github.com/Jalle19/xbmc-video-server/commit/1f9025c9959fbd63734bfe5f1066ce15a96e338d#commitcomment-13577135>.


Yes, I will change it to build*/ that is the standard name for most cmake build directories.


oops. Missed this regression.


Thank you, there look good....


So would we just add them to main.html and elements/html, or is something more complicated going on? Alternatively, we could leave this as is until 1.9, which adds a setting for exactly this purpose.


This check looks ok, but how does it look on GUI side?


Here's a screenshot of the GUI so far

![screen shot 2015-10-03 at 4 14 33 pm](https://cloud.githubusercontent.com/assets/816150/10264952/064f2490-69eb-11e5-9049-e6c1d6499de5.png)


I don't really like how this exposes the RevisionCollection to the world. So far we've one so much to not expose the TFS objects to the rules, and this breaks it. I ran into the same issue with the RateLimiter. where I decided, since it's internal, it would not be so bad.

I'd personally revert this change, or wrap the RevisionCollection and Revision and RevisionFieldCollection as per the other wrapped types. Though RevisionFieldCollection could probably use the same IWorkItemFieldCollection interface at least.

So far I haven't had the need to expose the revisions to the rules, unless someone is trying to do some kind of trend calculation or infinite-loop prevention...


I like this way alot more then just relying on the `{{excerpt}}`, nice work @nstapelbroek


`scope.$on('$destroy', onScopeDestroy )` is called before the  `element.on('$destroy'...)`.
Leaving this did not appear to hurt and I worry about conditions were the latter would not be called...
@jelbourn: are you aware of any issue with doing the `backdrop.remove()` **only** in the `onScopeDestroy` handler ?


We really *should* keep the username in the project, because eventually we're going to want that "Fred's version of Mary's project" stuff for web pages of remixes.


Daisy daisy who's your mama?


this is a test


better comment: "@param obj Object to be hashed and added to HashTable."


A more universal solution would look like this:

    target_view.location.href = "data:attachment/file" + base64Data.slice(base64Data.search(/[,;]/));

This solution is compatible with any form of data URI emitted by `readAsDataURL`, even if it's not base64, or if it includes an optional `;charset=` parameter.


Maybe this `30000` can be extracted as well into a meaningful constant.


Yeah there's an old issue about project history in the XML. Though really for the website this stuff belongs in a DB not just the XML.

--
Michael Ball
From my iPhone
michaelballphoto.com

> On Oct 3, 2015, at 1:27 PM, Brian Harvey <notifications@github.com> wrote:
>
> We really should keep the username in the project, because eventually we're going to want that "Fred's version of Mary's project" stuff for web pages of remixes.
>
> —
> Reply to this email directly or view it on GitHub.
>


This is a strange way to break the while loop, but it works.


(Excuse me for sending the message below to everyone, but GitHub broadcasts all comments to commits over the repository collaborators list. I don't understand how to fix it yet — P. K.)

Hi VadimRT

Please make commit messages more clear to the readers. For example, "Add my name to members.txt file".
Also, what tool or flow (list of git commands) have you used? The merge seems to be ill-formed.


variable names should be lowercase


You forgot to import TFM_AdminList, Also when importing +import java.util.Arrays and
+import java.util.List remove the extra +'s. There should be no +'s in  the code. Also please don't make a new PR. Just edit this one like this: Just click Files Changed at the top. (this: https://www.screenshottr.us/v/838410cce16e57cee11122d6befcdad0/1df59f104735881cc0996b9201b5ba61.png ) and then click this: https://www.screenshottr.us/v/e9bf98c469403b29cbcfeccaa0a26a03/5d7754e2004132b2e89a3e2d76e675e8.png ) make ur changes and commit it, Don't make a new pull request.


This does not actually rehash each object. It keeps each object in the same location. They should all be rehashed based on the new size of the table.


Из-за внешнего отступа у body получилось немного криво. Оказывается папка "css" прописана в .gitignore, поэтому normalize.css не загрузился. Ну да ладно, в следующий раз поправлю.


thanks Mike


commit message +1000


This line's a bit long, have you considered splitting it across multiple lines?


+1 to that.


The Raspidac3 overlay is already enabled in the RPi/RPi2 linux support patch.

The Raspidac3 config option is already enabled in linux.arm.conf.

What else do you think needs to be enabled?


I think I didn't commit any

On Sat, Oct 3, 2015, 8:43 PM Christoph Meißner <notifications@github.com>
wrote:

> OMG!
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/volunteer-planner/volunteer_planner/commit/8a5617a1facc2bba3aa12df58f8734c90a53b1a7#commitcomment-13576761>
> .
>


Currently these are here: https://github.com/RPCS3/rpcs3/blob/master/rpcs3/stdafx.h#L63


It may make sense to avoid using pandas in the examples unless we need it.  I think it is only being used to display a table of numbers.  It may make sense to look at Appendix B of my thesis as it includes an edited version of Kellie's notebook.  It may not be worth looking at (I can't remember how much editing I did).


Hi Terry,
I just committed this fix to the codebase.
thanks,
-brady


If things were in arrays, this could be done with for loops


Incorrectly branched from master before making changes. I have since deleted this branch and made the changes from the staging branch as I should have.


More for loops if things were in arrays


might be more readable if this was hidden away in a function called "ToggleLED"


It defaults to True rather than 1, but yes, the default is to enable it.


Thanks, just works fine for me now. :)


@mbootsector: Please do *not* stop the 600s test because of the noise in the forum!


This commit seems to block all plugins/scripts/bookmarklets that aren't approved. Is there an approval process, now? I see that dubx included themselves as approved.


this function could accept just the single array of structs, which could be setup to work for any number of scales


Again, with arrays, this could be made more readable


Je trouve dommage de hardcoder le délai de 30 jours : nombreuses sont les factures que nous transmettons avec 3 mois d'avance.


why don't you set currentMillis to millis() at the top of this function?


TeamCity Gensokyo Lobby :: Lobster Server [Build 0.0.1.36](https://pixelated.design/build/viewLog.html?buildId=143&buildTypeId=GensokyoLobby_LobsterServer&guest=1) outcome was **SUCCESS**
Summary: Running Build time: 00:00:09


what happens on timer overflow?


maybe have a function that prints all of this to the screen since this appears throughout the code


You can use the code for that purpose if your points are in pcd format. I think there's a a tool for conversion from some formats in the page of PCL.
Cheers.
Rodrigo.


Hmmm not keeping commit history, really should be giving credit where its due...

https://github.com/mythos234/TornadoRX-5.1-H815/commit/a8f3c8def135cb4b21942a6a3bfd17c891582a6f


just here to give a trigger target


all hell breaks loose without this!


and I couldn't figure out a dependency that would force a check of the tick box state when the `inst_points` are changed


seems to work


//Author Matin Munirs
//date Sep 24th 2015 version 1.0

class Account{

public:
    //explicit constructor
    explicit Account( int accountBalance );
    //getter method for balance
    int getBalance() const;
    //credit method deposit to account
    void credit( int amount );
    //debit method withdraw from account
    void debit( int amount );
private:
    //data member
    int balance;
};


// Assignment 1 Solution: ex03_12.cpp
// Create and manipulate Account objects.
//
// Bob Langelaan
// Oct. 13, 2006

#include <iostream>
using namespace std;

// include definition of class Account from Account.h
#include "Account.h"

// function main begins program execution
int main()
{
    Account account1( 50 ); // create Account object
    Account account2( 25 ); // create Account object

    Account account3( -25 ); // attempt to initialize to negative amount;

    // display initial balance of each object
    cout << "account1 balance: $" << account1.getBalance() << endl;
    cout << "account2 balance: $" << account2.getBalance() << endl;

    int depositAmount; // stores deposit amount read from user

    cout << "\nEnter deposit amount for account1: "; // prompt
    cin >> depositAmount; // obtain user input

    cout << "\ndeposit " << depositAmount
    << " into account1 balance\n\n";
    account1.credit( depositAmount ); // add to account1

    // display balances
    cout << "account1 balance: $" << account1.getBalance() << endl;
    cout << "account2 balance: $" << account2.getBalance() << endl;

    cout << "\nEnter deposit amount for account2: "; // prompt
    cin >> depositAmount; // obtain user input

    cout << "\ndeposit " << depositAmount
    << " into account2 balance\n\n";
    account2.credit( depositAmount ); // add to account2

    // display balances
    cout << "account1 balance: $" << account1.getBalance() << endl;
    cout << "account2 balance: $" << account2.getBalance() << endl;

    int withdrawalAmount; // stores withdrawal amount read from user

    cout << "\nEnter withdrawal amount for account1: "; // prompt
    cin >> withdrawalAmount; // obtain user input

    cout << "\nattempting to subtract " << withdrawalAmount
    << " from account1 balance\n\n";
    account1.debit( withdrawalAmount ); // try to subtract from account1

    // display balances
    cout << "account1 balance: $" << account1.getBalance() << endl;
    cout << "account2 balance: $" << account2.getBalance() << endl;

    cout << "\nEnter withdrawal amount for account2: "; // prompt
    cin >> withdrawalAmount; // obtain user input

    cout << "\nattempting to subtract " << withdrawalAmount
    << " from account2 balance\n\n";
    account2.debit( withdrawalAmount ); // try to subtract from account2

    // display balances
    cout << "account1 balance: $" << account1.getBalance() << endl;
    cout << "account2 balance: $" << account2.getBalance() << endl;

    system("pause"); // To allow us to view the output before the console window dissappears

    return 0; // indicate successful termination

} // end main


Thanks for the explanatory note.

For my reference, did you encounter a live situation where __package__ was None (just so I have a specific idea of where/when this happens)?


So cute :heart_eyes:


Raspidac3 is still not supported in the last official distro 5.95.5.
Hence it's in your 6.0 test release, guess it will be in the next official
distro.

---------- Původní zpráva ----------
Od: MilhouseVH <notifications@github.com>
Komu: OpenELEC/OpenELEC.tv <OpenELEC.tv@noreply.github.com>
Datum: 3. 10. 2015 22:40:29
Předmět: Re: [OpenELEC.tv] linux: update to linux-4.1.9 (d770c26)

"

The Raspidac3 overlay is already enabled in the RPi/RPi2 linux support
patch.

The Raspidac3 config option is already enabled in linux.arm.conf.

What else do you think needs to be enabled?

—
Reply to this email directly or view it on GitHub
(https://github.com/OpenELEC/OpenELEC.tv/commit/d770c269d4b685a10143c53200d814877dc469d5#commitcomment-13577228)
.

"=


This may overflow. In our calloc implementation we're dealing with such possibility.


The `super` implementation shouldn't be checking for `id`; strictly speaking the base BSO format doesn't require it. Will fix.


@TetyaseDaWina `getHealthPoints()` et `takeAKnock()` sont déjà définies dans `Character` :confused:


Hi @Jalle19,

what’s the difference for you between “Power Off” & “Shutdown”. The one is the server where is the backend and the other one is the Kodi software ?


I'm not sure about recalloc (since it's not standard), but realloc has interesting _feature_ where it behaves as plain malloc if p is NULL.

I think you should handle this case too.


:+1: nice work. The primary challenge of this homework was to learn how to use UITableView to manage a complex form, establish communication between view controllers, and apply the correct Auto Layout constraints. For each point in the homework feedback checklist, I'll either confirm that you're on the right track with the implementation, or I'll provide some feedback:

* For the restaurant cell, note that the subviews should be private.
* Great work on restaurant updates handled by the Swift property observer.
* Note that you should resolve all the Auto Layout constraint warnings in your nibs.
* Nice work linking the filter model with the controls. See this [popular form library](https://github.com/nicklockwood/FXForms) for an alternative to implementing forms manually.
* Nice work connecting the view controllers via the custom filter delegate. This is a popular pattern for handling events between view controllers.

This was a challenging assignment, congrats on completing it successfully!

-- dirk


This doesn't seem like a good design decision. If you see something other than what you expect, shouldn't you throw an exception rather than silently failing and moving on?


Amusingly, it already is; just commented out. I'll use it.


Both Raspidac3 changes are in the openelec-6.0 branch, added 20 Sep so not in the 5.95.5 release but will be in whatever comes next.


interesting..... I am typically not interested in using RestXQ, some issues pop up every now and then. What about <!-- putting this into a comment block --> ?


Hm. So after thinking about this more I think it is still wrong.

The issue is that prev_size can be (and usually is) larger than what application actually allocated. Since we round size up to size class.

Which means that this code will copy some amount past what application actually allocated. Which might be non-zero, which violates contract of recalloc.


Nice ;) Will be good when this is working properly.


Yeah, the ajv api -X command had been out of commission for a while.

The source of the problem was that experimental.__package__ is None:

Python 2.7.8 (v2.7.8:ee879c0ffa11, Jun 29 2014, 21:07:35)
[GCC 4.2.1 (Apple Inc. build 5666) (dot 3)] on darwin
Type "help", "copyright", "credits" or "license" for more information.
>>> import abjad
>>> abjad.__package__
'abjad'

Python 2.7.8 (v2.7.8:ee879c0ffa11, Jun 29 2014, 21:07:35)
[GCC 4.2.1 (Apple Inc. build 5666) (dot 3)] on darwin
Type "help", "copyright", "credits" or "license" for more information.
>>> import experimental
>>> experimental.__package__ is None
True

On Sat, Oct 3, 2015 at 5:03 PM Josiah Wolf Oberholtzer <
notifications@github.com> wrote:

> Thanks for the explanatory note.
>
> For my reference, did you encounter a live situation where *package* was
> None (just so I have a specific idea of where/when this happens)?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Abjad/abjad/commit/bad145aba03f8d06f8dc24251b47303d05c4ff85#commitcomment-13577317>
> .
>


Sorry for delay. I haven't seen notification from your update.

So as noted above, I now believe that there is no way we can meaningfully support recalloc at all. I should have understood this sooner. Sorry for that.


@TetyaseDaWina Je pense qu'il faut éviter de redéfinir les attributs définis dans les super classes - en l'occurrence `maxHealthPoints` et `weapon` - sinon tu perds l'avantage de l'héritage, par exemple si tu as besoin d'apporter une modification sur tes attributs. Pour régler le problème tu peux les initialiser dans le constructeur.


@ahrens couple things:

Any word on the analysis of this?   A straightforward send/receive (in particular, not incremental) leads to a microscopic but real amount of difference in data on a (read-only) clone of the send side versus a (read-only) clone of the receive side on their respective pools.

Also, (unrelated, but hopefully interesting to you) I've pulled in https://reviews.csiden.org/r/246/ to my local tree since it's a tiny change and I have lots of 1m recordsize datasets, and the performance change is very nice.


Rename to `ClearNotificationGroup` if it is an async method


printf(“Hello Holberton\n”);


Must be an `abstract` method if there is compilation condition


Nice, Glad you like it. Would you like me to create a pull request for this
commit to merge back into your repository?

On Sat, Oct 3, 2015, 22:25 Daniel Tamkin <notifications@github.com> wrote:

> I like this way alot more then just relying on the {{excerpt}}, nice work
> @nstapelbroek <https://github.com/nstapelbroek>
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/nstapelbroek/placidity/commit/a7d362f730edad2d1bad11c11fec52d5afe2afc2#commitcomment-13577189>
> .
>


Why did you reformat again?


Hi. If you want, you can add some tests and submit a PR and I'll merge this functionality into master :-)


In my current implementation, the `id` is not unique in many cases, so add an optional parameter `group` to the method.


yeah, makes sense. let me read it more carefully and get back to it.


The tag `Modern Gitter` should be a Constant, check the `Configuration` project


                                                                                I don't mind sharing subscription too. I hope we can hear from Shani.                                                                                From: moe1999Sent: Saturday, October 3, 2015 3:53 PMTo: Shani-08/ShaniXBMCWorkReply To: Shani-08/ShaniXBMCWorkCc: aljudehSubject: Re: [ShaniXBMCWork] Update yoUrls.xml (d5ce52a)Very generous buddy. You should ask shani and he'll make a decision.

Sent from my iPhone

> On Oct 3, 2015, at 3:52 PM, allouch89 <notifications@github.com> wrote:

>

> I would to make the next payment for the subscription

>

>

>

> > Den 03/10/2015 kl. 20.35 skrev moe1999 <notifications@github.com>:

> >

> > We all are. It's a matter of kindness.

> >

> > Sent from my iPhone

> >

> > > On Oct 3, 2015, at 2:34 PM, aljudeh <notifications@github.com> wrote:

> > >

> > > Oh serious. So why we all not contribute in payment? He is doing a great job! From: moe1999Sent: Saturday, October 3, 2015 2:32 PMTo: Shani-08/ShaniXBMCWorkReply To: Shani-08/ShaniXBMCWorkCc: aljudehSubject: Re: [ShaniXBMCWork] Update yoUrls.xml (d5ce52a)Yes. I think this is now everyone is getting Arabic tv. Thanks to shani

> > >

> > >

> > >

> > > Sent from my iPhone

> > >

> > >

> > >

> > > > On Oct 3, 2015, at 2:31 PM, allouch89 <notifications@github.com> wrote:

> > >

> > > >

> > >

> > > > What membership? Did someone pay something so everyone else can watch?

> > >

> > > >

> > >

> > > >

> > >

> > > >

> > >

> > > > > Den 03/10/2015 kl. 20.21 skrev moe1999 <notifications@github.com>:

> > >

> > > > >

> > >

> > > > > It seems like the membership has been finished.

> > >

> > > > >

> > >

> > > > > Moe

> > >

> > > > >

> > >

> > > > > —

> > >

> > > > > Reply to this email directly or view it on GitHub.

> > >

> > > > >

> > >

> > > > —

> > >

> > > > Reply to this email directly or view it on GitHub.

> > >

> > > >

> > >

> > >

> > >

> > > —Reply to this email directly or view it on GitHub.

> > > —

> > > Reply to this email directly or view it on GitHub.

> > >

> > —

> > Reply to this email directly or view it on GitHub.

> >

> —

> Reply to this email directly or view it on GitHub.

>

—Reply to this email directly or view it on GitHub.


@TetyaseDaWina Pour les packages de tests unitaires il vaut mieux utiliser les mêmes noms que ceux utilisés dans le dossier `src`. Ça demande moins à réfléchir et c'est comme ça qu'on faisait au S4. Ça pourrait aussi permettre à des outils d'automatisation de générer les sources à partir des tests par exemple (c'est juste une intuition).


There's a problem with this line.


Create a pull for your most recent commits and ill merge them in right now :smile:


TeamCity Colore :: Main Build [Build 165](http://tc.sharpblade.net/viewLog.html?buildId=708&buildTypeId=colore_mainbuild) is now running


This commit title is wrong.  should simply read "cleanup"


pos2d_to_pos3d() and magic number 1.48


TeamCity Colore :: Main Build [Build 165](http://tc.sharpblade.net/viewLog.html?buildId=708&buildTypeId=colore_mainbuild) outcome was **SUCCESS**
Summary: Tests passed: 111 Build time: 0:1:52


表達數字增減的地方統一一下，看要用哪種，其它幾處也一樣
1. 攀登需消耗2點AP。\n戰鬥中增加1點防禦。
2. 攀登需-2點AP。\n戰鬥中+1點防禦。


TeamCity Colore :: Main Build [Build 166](http://tc.sharpblade.net/viewLog.html?buildId=709&buildTypeId=colore_mainbuild) is now running


No need for a username


TeamCity Colore :: Main Build [Build 166](http://tc.sharpblade.net/viewLog.html?buildId=709&buildTypeId=colore_mainbuild) outcome was **SUCCESS**
Summary: Tests passed: 111 Build time: 0:1:47


Everybody puts whatever they want in the claims


Interestingly, http://techpubs.sgi.com/library/tpl/cgi-bin/getdoc.cgi?coll=0650&db=man&raw=1&fname=/usr/share/catman/p_man/cat3x/mallinfo.z says that recalloc can only handle objects that were previously allocated via calloc or recalloc. Which work for your implementation (after some fixes).

But MS documentation (https://msdn.microsoft.com/en-us/library/ms235322.aspx) explicitly says that p could be returned my malloc or realloc. And that seems to be too strong requirement. Which we cannot meet AFAIK.


Program description?


Oh, okay. I'll do it when I can use a computer, thank you.

On Sat, Oct 3, 2015 at 3:34 PM, Kevin Musiorski <notifications@github.com>
wrote:

> you'll need to store this in a variable called hours or something similar
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/AntuanLewis/ProgrammingClass2015-2016/commit/92e33caf7b7326dc9f35e9dc4a054840a54fda73#commitcomment-13576999>
> .
>


Thinking a bit more, I think having restricted implementation (which only handles pointers returned from calloc or recalloc) is better than nothing.

So ignore my comment about "impossible", but do fix NULL case and please handle overflow.

Also I think you can try to always handle it via realloc.


@TetyaseDaWina Le mock ici n'a aucun intérêt, il fait exactement la même chose que la classe mère :confused: On a discuté un peu des mocks avec le prof au dernier TD. Les mocks servent à limiter les interactions entre la classe tester et les classes dont elle dépend. Par exemple, imagine que tu doives tester `Room`. Tu sais que `Room` dépend de `Door` cette classe `Door` n'a pas encore été livrée. Dans ce cas tu vas créer une classe `DoorMock` qui fournit les mêmes fonctionnalités pour la remplacer. À quelques détails près c'est ça (c'est ce que j'en ai compris en tout cas).


Thank you and yes it's french


@TetyaseDaWina Tu vois ici tu aurais mis `new Character(...)` ça aurait produit exactement le même résultat.


better? better.


al contrario mano, gracias por contribuir!


I was actually working on that. THis direct code pass tests but I have to check that interaction with bind and quote work slike in MPL.

I believe it's correct but we may need deeper test


Seems like this is not possible because the interface does not return the view that called us so we can't switch on the views. I merged the OnClickListeners but I'll be keeping this one :)


These `style="color:white"` styles don't seem to have any effect? It seems like the rules in `colorNested > div` do this already?


agreed. obselete configuration.


In fact, no, it's sufficent. Merging this


Just ran this locally for the first time, looks great man. Let's tidy it up and post it. I still have the domain.


At the moment, it should be the first behavior.
Vld should report about before unloading last exe/dll that use it.

> this behavour could lead to crashes, if the closed source application happens to cache function addresses after vld was loaded and then call these functions after vld was unloaded.

I think main issue that we do not unhook functions at unloading vld.


@keith Good to know. That might explain why `ghu` stopped working for Basecamp


Should this be final?


> Agreed, but it was already included in the repo. look at lines 694-706 above.
> maybe it was included in error? or maybe there is a specific reason?

It's probably original author code.

> i suggest we remove the windows and system directories, as it increases the symbol search time exponentially

Agree, we should try remove and check that loading symbols okay


No, this doesn't ring any bells, and it does look odd given that it's nullable.

I vaguely remember something about `required` being a good idea when the dictionary argument itself is required, since a "required" argument that will accept just `{}` as an argument is silly. However, that isn't the situation here.


If you want to use this in your final version, you'll need a prime number generator.


Good.


Ok, if it not cause performance regression.


??? are you adding executable scripts to movieclip frames???


I think so. I don't plan on having a function that can change it at this time and I can even make it public if anyone wants to be able to know what the load factor is. If at some point the load factor needs to be mutable then I can make it not final.


Uses the code? Hashcode?


Put imports above header comment.


Should this be static and final?


Is this necessary? When would spot be negative?


TeamCity Colore :: Main Build [Build 167](http://tc.sharpblade.net/viewLog.html?buildId=710&buildTypeId=colore_mainbuild) is now running


or, != null?


Update for member page having corrected grid


`GetModuleHandleEx` calls can lead to additional ldrLoadLock and cause performance regression.
We need at least simple test that wouldn't work with old code


Why would you reset occupied?


TeamCity Colore :: Main Build [Build 167](http://tc.sharpblade.net/viewLog.html?buildId=710&buildTypeId=colore_mainbuild) outcome was **SUCCESS**
Summary: Tests passed: 111 Build time: 0:1:46


По возможности не удаляй что-то, что ты не добавлял, без необходимости.


TeamCity Colore :: Main Build [Build 168](http://tc.sharpblade.net/viewLog.html?buildId=711&buildTypeId=colore_mainbuild) is now running


I appreciate the effort here, though there may be a more efficient way of generating another prime number.


Импорты сгруппированы по названию пакетов, поэтому эти два логичнее перенести к остальным.


You should just develop a "getNextPrime" object - or steal (and cite!) one from somewhere.


I copied and pasted the code in from text wrangler and the int x = 1 and x = x*2 lines were commented out in my version but somehow the commented out didn't transfer onto github


Optionally, the calculated ugly numbers could be cached (stored in an array or map) so that subsequent calls with a k less than the current one could be immediately looked up, and this could also be used to avoid recalculating the first part of the sequence even in case k is greater than a previous one.


This should probably be private.


TeamCity Colore :: Main Build [Build 168](http://tc.sharpblade.net/viewLog.html?buildId=711&buildTypeId=colore_mainbuild) outcome was **SUCCESS**
Summary: Tests passed: 111 Build time: 0:1:47


Переименуй `dto` в `addImageForm`.


Great work @istarkov 💪


Не удаляй просто так.


Thanks!


Nice. Yeah later it occurred to me that using the svg logo would make this
a lot simpler, look better on phone, etc.

Have any bright ideas about color picker ui? On hard mode we could pop open
the native color input thing for infinite color options.

On Saturday, October 3, 2015, Justin Rodermond <notifications@github.com>
wrote:

> Just ran this locally for the first time, looks great man. Let's tidy it
> up and post it. I still have the domain.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/stereosteve/what-color-is-google/commit/69533734a5f34068d428f30d836181ee98b7f13c#commitcomment-13577411>
> .
>

--
Steve


Похоже, что ты использовал пробелы вместо табов для отступов. Надо настроить IDE, чтобы использовала табы вместо пробелов.


Tests should be deterministic, when u are using any random function... you know.


i think something in this breaks sab..  startup sab.. add nzb.. working .. in glitter set speed limit.. instantly see all downloads just hang.. speed then dies off in bps meter.. cant shutdown/restart sab via gui either. prob stuck in some loop..

what logs show:
```
2015-10-03 16:55:28,948::DEBUG::[decoder:103] Decoding Part72of137.64240427834F418482F751D0DA91EAAF@1398531994.local
2015-10-03 16:55:29,072::DEBUG::[decoder:103] Decoding Part83of137.FA19222000B14842B08DC991CD8E4442@1398531994.local
2015-10-03 16:55:29,091::DEBUG::[decoder:103] Decoding Part88of137.40A1720278D247729C0DB360BE651D69@1398531994.local
2015-10-03 16:55:29,151::INFO::[downloader:293] Bandwidth limit set to 91
2015-10-03 16:55:31,703::DEBUG::[bpsmeter:271] bps: 6187853.74672
2015-10-03 16:55:36,753::DEBUG::[bpsmeter:271] bps: 2265068.24952
```

only way to fix is to kill sab process or issue control+c on console and start sab back up.


Comment


TeamCity Colore :: Main Build [Build 169](http://tc.sharpblade.net/viewLog.html?buildId=712&buildTypeId=colore_mainbuild) is now running


Related: #9969 and #9971.


Скорее всего, лучше использовать `TestObjects.createSeries()` вместо ручного создания. В этом случае, наш объект будет более-менее заполнени каким-то тестовыми данными.


Насколько я помню, в файле комментарии начинались с заглавной буквы.


\o/


TeamCity Colore :: Main Build [Build 169](http://tc.sharpblade.net/viewLog.html?buildId=712&buildTypeId=colore_mainbuild) outcome was **SUCCESS**
Summary: Tests passed: 111 Build time: 0:1:47


@evilhamster thanks for your testing work. Changes to SetFromAblum still has not fixed the lack of artist names on the albums lists, but I am looking into it further. Suggestions welcome.


It seems that the wrong plot is being displayed. Should be (images/count4.png) instead?


Отлично сделал три теста.

Дополнение: придерживайся того же стиля, который уже используется в файле (т.е. между when/then нет лишней разделяющей строки, вместо этого все команды сдвинуты вправо).


TeamCity Colore :: Release Build [Build 80](http://tc.sharpblade.net/viewLog.html?buildId=714&buildTypeId=colore_releasebuild) is now running


This line looks like a duplicate of the line above. Is that intentional?


TeamCity Colore :: Release Build [Build 80](http://tc.sharpblade.net/viewLog.html?buildId=714&buildTypeId=colore_releasebuild) outcome was **SUCCESS**
Summary: Skipping master branch Build time: 0:0:10


TeamCity Colore :: Release Build [Build 81](http://tc.sharpblade.net/viewLog.html?buildId=715&buildTypeId=colore_releasebuild) is now running


Nah I love how you did it. Click through, no color picker, forces users to cycle through all colors and consider them. It doesn't have to be a "minimal clicks" UI b/c most people will only do it once.


the transparent png is clever too, works fine but yea we need to consider high density phones


Nope. I fixed that in https://github.com/pydanny/cookiecutter-django/commit/ab62fdb4acbf0bfcb84431127f10f631a923edd0


Did you mean to include this file? If not you can store a copy elsewhere and then hit `git rm path/to/file` and add another commit.


2 warnings left


В целом -- отличное начало. Помимо уже перечисленного нужно добавить еще тестов для проверки следующего:
- сервис вызывает метод `ImageService.save()`
- сервис вызывает метод `seriesDao.save()`
- сервис добавляет `Image` в `Series`


reverted git before this commit, `git.exe reset --hard 189dca1e762bc857a4739bf4fb7a253579bf7a18 --` setting speedlimit in glitter is fine. so confirmed issue is in this commit.


`&& \` <- the `\` here is obsolete syntax


@bowmanat


You can use `c.freshName` to generate reasonably unique names, and then you can pass them around, unquoting into trees as necessary.


TeamCity Colore :: Release Build [Build 2.2.2.81](http://tc.sharpblade.net/viewLog.html?buildId=715&buildTypeId=colore_releasebuild) outcome was **SUCCESS**
Summary: Tests passed: 111 Build time: 0:3:42


`|| die`?


missing `|| die`s


I'm not happy with this system. CommandManager doesn't even manage anything, it just executes commands. Why are commands handled in the Command class? This along with events confuses me.


@amjith, are you aware of #353? Want to weigh in? Or submit a pull request?


all this myconf stuff is very difficult to read, why not use a bash array?


We need to detect when current tab loses focus so we can warn the cheaters that their results have been placed into the suspect category.


Good. I suppose it is center aligned to the div? Can we get it centered in the browser window (currently its kind of 15% to the left).

We still have the raw templates "flashing" in the back ground though.

________________________________
From: svoinea <notifications@github.com>
Sent: Saturday, October 03, 2015 9:44 AM
To: informatics-isi-edu/chaise
Subject: Re: [chaise] Added a spinner like in the detail page while loading the search page. (c28a390)

Try my latest https://serban.facebase.org/chaise/search/

On Fri, Oct 2, 2015 at 10:11 PM, Jessie Wong <notifications@github.com>
wrote:

> Loading this commit on my VM makes the sidebar invisible even after the
> page is done loading. [image: :cry:]
>
> [image: screen shot 2015-10-02 at 10 09 53 pm]
> <https://cloud.githubusercontent.com/assets/4585431/10261551/61823274-6952-11e5-8720-8e83932670ce.png>
>
> -
> Reply to this email directly or view it on GitHub
> <https://github.com/informatics-isi-edu/chaise/commit/c28a390af3ec37ef3fe8581afa4cfe9a6bb3abe2#commitcomment-13574383>
> .[image: Web Bug from
> https://github.com/notifications/beacon/ABiPaerC8wStMbCazsMPb970GgRVeg5aks5o31sQgaJpZM4GIX0u.gif]
>

-
Reply to this email directly or view it on GitHub<https://github.com/informatics-isi-edu/chaise/commit/c28a390af3ec37ef3fe8581afa4cfe9a6bb3abe2#commitcomment-13576391>.

[https://avatars1.githubusercontent.com/u/1609577?v=3&s=200]<https://github.com/informatics-isi-edu/chaise/commit/c28a390af3ec37ef3fe8581afa4cfe9a6bb3abe2#commitcomment-13576391>

Added a spinner like in the detail page while loading the search page. · informatics-isi-edu/chaise@c28a390 · GitHub
chaise - An adaptive user interface to the ERMrest service.
Read more...<https://github.com/informatics-isi-edu/chaise/commit/c28a390af3ec37ef3fe8581afa4cfe9a6bb3abe2#commitcomment-13576391>


Actually, it might be better to remove the GEOFENCE_TRANSITION_DWELL and .setLoiteringDelay(), it doesn't seem that the feature really brings anything to the user experience ... just spams with more notifications :)


Very nice.


Big error here, setName allows plugins to change the real name of the player that will affect the getPlayer() method in Server.java


@return


Was this your final version of HashTable version 0? Seems incomplete.


Ever since this commit, Travis segfaults on the first OpenCL format (sha1crypt-opencl).


Description?


@TetyaseDaWina Ici j'aurais passé la cible du lapin en paramètre, comme ça: `rabbit.attack(hero)`. Je sais pas si c'est une bonne pratique mais ça a le mérite de se lire comme en français, du coup ça paraît plus naturel. Imagine toi au milieu d'une foule avec ton lapin et tu lui dis "lapin! attaque!", le lapin te répondrait probablement "lequel?!" :smile:


{/$$}, not {\$$}


Hmm. I guess we'll call this HashTable pre-Alpha.


Built-in deep equality checks are nice, but can be costly. It's unclear how can the identity of arrays and objects be compared. Would the `===` operator be encouraged, since this is JavaScript?.


Is this not your updated code for version 0? Seems incomplete.


This is version 0 I assume?


This seems incomplete. Is it your latest version of Version 0?


I think this code and from e6a98e1bf295fa480e26682ca96f1b1bae9b58f7 cause performance degradation.

Before your changes:
```
Dynamic CRT
[ RUN      ] ResolveVal/DynamicLoader.Thread/0
All threads finished correctly.
[       OK ] ResolveVal/DynamicLoader.Thread/0 (124 ms)
[ RUN      ] ResolveVal/DynamicLoader.Thread/1
All threads finished correctly.
[       OK ] ResolveVal/DynamicLoader.Thread/1 (5306 ms)

Static CRT
[ RUN      ] ResolveVal/DynamicLoader.Thread/0
All threads finished correctly.
[       OK ] ResolveVal/DynamicLoader.Thread/0 (416 ms)
[ RUN      ] ResolveVal/DynamicLoader.Thread/1
All threads finished correctly.
[       OK ] ResolveVal/DynamicLoader.Thread/1 (2438 ms)
```
After:
```
Dynamic CRT
[ RUN      ] ResolveVal/DynamicLoader.Thread/0
All threads finished correctly.
[       OK ] ResolveVal/DynamicLoader.Thread/0 (144 ms)
[ RUN      ] ResolveVal/DynamicLoader.Thread/1
All threads finished correctly.
[       OK ] ResolveVal/DynamicLoader.Thread/1 (5356 ms)

Static CRT
[ RUN      ] ResolveVal/DynamicLoader.Thread/0
All threads finished correctly.
[       OK ] ResolveVal/DynamicLoader.Thread/0 (581 ms)
[ RUN      ] ResolveVal/DynamicLoader.Thread/1
All threads finished correctly.
[       OK ] ResolveVal/DynamicLoader.Thread/1 (41942 ms)
``


Ага, отлично.

Когда ты делаешь коммит, который относится к какой-то задаче, то ты можешь в комментарии к коммиту написать специальную фразу "Fix #NN" где NN это номер этой задачи и тогда тикет будет закрыт автоматически, также в нем появится ссылка на твой коммит.

Сейчас ты исправил, но не закрыл задачи. В этот раз тебе придется закрывать их вручную.


Your variable names don't tell me much. Also, do you really need all these variables?


addition comment on line 12


display comment line 6 deletion line 10 addition


Must have missed the api. Will fix tomorrow.


Вижу, что заодно и файл отформатировал? В целях чистоты и упрощения ревью в следующий раз лучше делать это в два коммита.


// FIXME: See if we can figure out a way to set this to 'AES-256-CBC' for new installs only


Javadoc?


А здесь-то зачем такое имя? Можно было уже по-проще назвать, например, `demo-app` каким.


@KindDragon: Based on current behaviour uncommenting this line will result in an 0xC0000005: Access violation exception


addition comment line 282


Has a pull request for this been created?   If so, why hasn't it been accepted?


@jmoudrik I'll check in a fix. Thanks.


whoops


I just tried running "--test-full=0 --dev=cpu" on my OSX laptop and see no problems.


Rehash seems like it would work, but is cryptic to read. Use better names and spacing.


I don't see a put method that takes in 2 parameters. Would this compile?


There is another commit below it.


Теперь аргументы `request`/`response` стали не нужны => можно убрать.
Метод больше не возбуждает исключения => можно поправить его сигнатуру.
Вместо возвращения `ModelAndView` можно возвращать простую строку с названием view-шки (нет смысла создавать более сложный объект без необходимости).


What is this doing?


@TetyaseDaWina `eat(String food)` c'est bien aussi :stuck_out_tongue_winking_eye:


Actual? I think it's an array.


Currently Squiggle provides a polyfilled implementation of `Object.is` available in scope as `is(x, y)`. I plan to move that into a binary operator like `x is y`.

Also, that reminds me, I should mention that speed is *not a goal* in Squiggle (at least not yet). I'm trying to make a language with ideal semantics and syntax, then optimize what I can, when I identify bottlenecks.


When reading the docs from the beginning, this is the first time `def` and `in` are used, and there's been no mention or description of them yet.


Well, not the "user" necessarily.


I would write it like this:

```js

    this._client.send(rpc.methodName("transactAndHold"), param, function (error, data2) {
      var
        newData2;

      // tendermint rpc results is prefixed with a type byte
      newData2 = Array.isArray(data2) ? data2[1] : data2;

      callback.apply(null, [error, newData2].concat([].slice.call(arguments, 2)));
    );
```


Why do you need absolute value here? And what is this doing exactly?


Not sure I understand displacement.


This won't work on other intermediate versions and implementing a way to handle it  thick defeats the purpose a little bit. So I'm wondering if using only the default rpi kernel version would be the right way to do in order to avoid complexity.


You shouldn't need 2 loops to accomplish a rehash.


please remove this new line.


Could you please squash this commit?


Yes, but that appears to be the next version?


Thanks for the fix, but this since the production server uses a separate `settings.py` this won't fix the problem. I'll update the one on the server.


It is. Why is it incomplete?


These were subsequently renamed to ``base64encode`` and ``base64decode`` respectively, in 0b85d35e87f26fd2ffdd241ed45a29f8277ab019.


Some of the methods are empty or unfinished.


nope


You have to open them by clicking on the expansion icon where the line numbers are.


Yeah sorry, haven't looked at this in ages... My version doesn't seem to take in scale factors for the normals... looking into that, will get back to you soon if it works


...这两天成迷2K16，没来得及reply。

我把“细枝末节”的修掉的主要原因是：细枝末节其实是一个带有贬义色彩的词汇，在正常的中文描述中你很难找到与这个词正面或者中性的联用的。 然而HTTP里虽然有很多琐碎的规定，但总还是有它的用途的..


To find minimum edit distance to modify from one string to another


`MEDIUM`


Sweet change.


@tomteman I updated to alpha 38, look at the [CHANGELOG](https://github.com/angular/angular/blob/master/CHANGELOG.md).
There are changes in the API, I fixed at our hand although the countdown component is not playing very well (you can see the seconds left the circle is invisible), I will fix it tomorrow.


because hash codes can be negative. like the hash code of "potato" is -982438873


"general methods"


It would be better to fix the warning here by backporting d2c1fce24e955d25f2645e827669e67372024135 from master to avoid gratuitous discrepancies between the branches.


""list(set = set,
      get = get,""


This commit seems to be in the wrong order somehow: surely it should come after 53a34f758a29dab94d8fddb04e6983d6b9af1564 which introduces `wxCLANG_WARNING_SUPPRESS` macro and not before it?


oh i see what i did wrong


我觉得这里的pause其实是有闲置的意思， 所以你觉得挂起怎么样？


Ah, got it. Silly GitHub.


I can only agree with the changes and guess you tested them?


FYI: We usually merge PRs and then update the changelog manually, to avoid any conflicts. :wink:

Welcome to the maintainers team btw.! :tada: :smiley:


`rendom` hours of sleep needed.


Yeah, better that way.


I have made changes and revisions to my code since then, but I have not
committed them to github just yet.

On Sat, Oct 3, 2015 at 3:32 PM, burkhartb <notifications@github.com> wrote:

> This won't work. You don't actually rehash any of the items. You just add
> them to a new temporary array and then erase table.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/aseth2017/HashTable/commit/e5863610cf94f2fe7150afbfc47a094a9d4ce9ce#commitcomment-13576994>
> .
>


@DjWarmonger on client-side tavern buttons were enabled in ally towns since forever, but attempt to buy hero fail because server don't allow that.


@lots0logs Well, at least we know what's wrong. You always have to look at the bright side...


Do we really need "normal" and `IF_VALID` versions? IMHO just having the latter should be enough, why would you ever want to disable a warning that might be unknown to the compiler? I.e. I'd rename `wxCLANG_WARNING_SUPPRESS_IF_VALID` to just `wxCLANG_WARNING_SUPPRESS`.


Yeah, I was just about to mention my (broken/untested) attempt to fix this in #9969. I've heard that we're supposed to default to yes here, so I too would love to know what's going.


nice catch


Looks good. My, you're unstoppable!


@shlevy Are the `CONFIG_` prefixes supposed to be there? I thought we handled that somehow.


Forgot it.


Haha we will never fully understand Github.


Looks good code-wise. This one needs extra-care when testing.


This probably doesn't need to be final, since in theory you could change it later.


I'm not sure ```#fafafa``` is the right color for this. It's almost impossible to distinguish between the background and the card. In the spec, they use ```#eeeeee```, which looks much better.


In the future, for dependency changes it would it be possible to do a larger version increment, like `2.1.0`? This change tacitly broke the build of a package that I had not yet upgraded to this new version. thanks!


Hmm, ok. :> Eager to see that in game.


What happens if it reaches the end of the array?


Alright, I see all your code now. Excellent job on Version 0!


BTW I still want to rewrite all this "#define" macro magic because it's make debugging way harder.


I tried inverting the normals, that only made the lighting upside-down, and I looked at your code and I couldn't see where you put in the negative scale factor :/


^ lol


Why is this OS X specific? Shouldn't we just always create an appropriately scaled `wxBitmap` for all platforms, including even MSW?


Coincido, es muy poco clsro toda esta inicializacion. Además faltan todos los casteos y máscaras


You don't really need to store percentFilled - you can just calculate that when you need it.


Don't initialize your private variables when you declare them. Do it in the constructor.


Guys, we now have what will likely be the 1.0 attack/def system. :>


Avoid +100. Do you mean +arr.length?


This parameter isn't used anywhere within this pull request. I don't see the use of it.


k, SF.


You shouldn't need 2 loops to rehash...


You can use `from . import model`


Nevermind, I figured out what was happening - I was only looking at your commit changes, not the entire file.


Use new version numbering scheme: https://github.com/OCA/maintainer-tools/blob/master/CONTRIBUTING.md#version-numbers


You can move this to a README.rst file


Nevermind; I was looking at your changes only, not the entire file.


And follow OCA's README template: https://raw.githubusercontent.com/OCA/maintainer-tools/master/template/module/README.rst


Remove this key


You can do this with only 1 loop...


Change it to 'data'


unnecessary indentation here.


Remove this editor specific line


The correct place for these is in (or by defining) the ``_storage_format_specification`` of the offending class, not in ``StorageFormatManager``.

```
@property
def _storage_format_specification(self):
    manager = systemtools.StorageFormatManager
    keyword_argument_names = manager.get_keyword_argument_names(self)
    if not getattr(self, 'no_resize', None):
        keyword_argument_names.remove('no_resize')
    return systemtools.StorageFormatSpecification(
        self,
        keyword_argument_names=keyword_argument_names,
        )
```


Why is population a double? Also, initialize variables in the constructor, not when you declare them.


You can remove this file


Use `from . import...` form


Unnecessary indentation.


You should add openupgrade to external_dependencies key and mention it on the README.


cms.locale?


eny link download ? !!!!


Does it deserve to update views to new standards?


Oh yeah I spaced on that because this commit required some manual work at the CLI to merge and so I wasn't thinking in terms of the web UI workflow. I just did another one like this before I saw your comment but I'll switch to updating the changelog as a separate commit from now on.

And thanks for the welcome! I'm sad that I didn't get to go to the dinner last Monday but hopefully we'll get another opportunity to all hang out at a future hashiconf. :grin:


This works but all mppt data will be the same. I would recommend having a `static value = 56.0f` and have each of the values be `value++`.


Your code always rehashes here - and why is there probing in your constructor?


Good catch!


Again a static variable might be nice to make all the CMUs different.


@Snaipe, on a second thought, would it make sense to have this sort of blocks refactored in some common header under `src/compat` or in `src/config.h.in` (along with `#include "common.h"`) for better DRY'ing?
Since -- for instance -- `strtok_r` is probably going to be used in other places in future (such as junit reporter).


Please add a blank line between these statements for readability


"wanna learn" or "want to learn"


I lol'd at this mistake too....


Only use @param and @return when there are parameters and a return type.


with this change, the statement in the readme.md @The functions return true if a value can be cast to the designated type without data loss, and false if it cannot@ , is no longer true..


Remove - getRect covers this


A line or two introducing this section and the camera object would be helpful for the new user.


  <target host="*.tor2web.com" />
  <target host="*.tor2web.org" />
  <target host="*.onion.cab" />
  <target host="*.onion.city" />
  <target host="*.onion.direct" />
  <target host="*.onion.link" />
  <target host="*.onion.nu" />

  <test url="http://thundersplv36ecb.onion.link/" />
  <test url="http://32rfckwuorlf4dlv.onion.link/" />
  <test url="https://dirre2hfssh2offv.onion.link/" />
  <test url="https://torlinkbgs6aabns.onion.link/" />
  <test url="https://deepdot35wvmeyd5.onion.link/" />

  <rule from="^(http|https)://([a-z0-9\.]*[a-z0-9]{16})\.onion\.(link|city|nu|cab|direct)/" to="http://$2.onion/" downgrade="1" />
  <rule from="^(http|https)://([a-z0-9\.]*[a-z0-9]{16})\.tor2web\.(com|org)/" to="http://$2.onion/" downgrade="1" />


Why do you need to total the ascii values?


Your rehash method needs to call put to actually rehash each item.


No idea what the underlying issue was. The _storage_format_specifier of ImageRenderSpecifier was correct.

The recent commits to revive the experimental API had the collateral benefit of fixing this problem.

Fixed in 2238bdc3502ffbb7edd2272d97dc11b121d336bf.


Daniil, thank you for pointing out the bug with the validation step. However, I don't see changes to the Podam implementation. If Podam doesn't handle the vector above, shouldn't something change in the implementation? Also, in future, could you please make pull request to the develop branch? Master is rebased only to released code.


Mmm wouldn't that would work only if cms is installed? Maybe I can change it to cms.locale in the new PR #479


Yep. It works with MinGW and MSVS2013. Note that I've explicitely used release dlls only. That's because that's what interesting IMHO. Now, it's easier to build a release ^^


Cool :)


An example calling sequence would be very helpful


yes ^^


Forgot about static variables, will do.


In addition to the minor in-line comments, an example calling sequence would be very helpful.


<!--
This ruleset ensures that Tor Browser Bundle users don't accidentally use a Tor2web node.
Wanted to include <target host="*.onion.to" /> but it wouldn't work due to a bug associated with the keyword "to".
Need to add Tor2web to the downgrade whitelist
-->

<ruleset name="Tor2web">
  <target host="*.tor2web.com" />
  <target host="*.tor2web.org" />

  <target host="*.onion.cab" />
  <target host="*.onion.city" />
  <target host="*.onion.direct" />
  <target host="*.onion.link" />
  <target host="*.onion.nu" />

  <test url="http://thundersplv36ecb.onion.link/" />
  <test url="http://32rfckwuorlf4dlv.onion.link/" />
  <test url="https://dirre2hfssh2offv.onion.link/" />
  <test url="https://torlinkbgs6aabns.onion.link/" />
  <test url="https://deepdot35wvmeyd5.onion.link/" />

  <rule from="^(http|https)://([a-z0-9\.]*[a-z0-9]{16})\.onion\.(cab|city|direct|link|nu)/" to="http://$2.onion/" downgrade="1" />
  <rule from="^(http|https)://([a-z0-9\.]*[a-z0-9]{16})\.tor2web\.(com|org)/" to="http://$2.onion/" downgrade="1" />
</ruleset>


It is already enabled by default for the entire db. If it's safe enough to enable by default (I only hear problems when people explicitly try to use restxq, not when they don't despite the default), why not carry the default over here too? The motivation here is that people like Harry find restxq "stops working" in eXide-created apps.


I've prepared something to show ^^


Hunh. Bizarre.


Oh dear, sorry, a dependency change alone shouldn't require a minor bump, but this one, of course, had a breaking change to accomodate the dependency bump. Should have been a major bump; I'll re-tag it.


Yes. But I've set everything to 0 so far. We need to decide how we really want to implement our triangle


FYI [vim-plug](https://github.com/junegunn/vim-plug) is quite a lot faster than Vundle.


Nice approach! Additionally, you may consider caching this array so that subsequent calls can make use of it.


3.0.0 released. I force pushed away 2.0.1, hopefully nobody else used it in the hour it existed.


I'd suggest searching for `OVR_KEY_IPD` and / or `OVR_KEY_EYE_HEIGHT`.  These are the two key values that developers really need to integrate into their engines since they impact the user experience the most.


@Bertram25 see here
http://forum.freegamedev.net/viewtopic.php?f=35&t=6576


:-1: please add ``- "./data:/data" instead (media files would then be in ``myproject/data/media`` by default.


Not that I know of, unfortunately, unless you implement your custom version of HashMap that returns a default value instead of null when the key is not present.


Might as well make it `local t = getTickCount() + 1`


It's not often that I see an ArithmeticException being thrown :)


In this case, maybe something like Multimap from the Guava library could be handy http://docs.guava-libraries.googlecode.com/git/javadoc/com/google/common/collect/Multimap.html


If you replace this line with
```
	//translator.translate();
	translator.getTranslatorObject(function(trans) {
		trans.doWeb(doc, url);
	});
```
then it is working for me. However, I don't understand why this should be anyhow different. Moreover, we have more translators calling the EM translator just with `translate()` and they seem to work...


good idea


BTW another example for multiple is http://www.sae.org/search/?content-type=%28%22PAPER%22%29&qt=effects&x=0&y=0


Agreed, using a trie seems more elegant in general, but in practice a HashSet would probably be sufficient. Thanks for pointing out anyway! In a real code review this would have been an interesting point to discuss.


I found that regression for Static CRT caused by 8041990def47964047328ccfc7837b43673b5a86


Nice!


It makes no sense to move it here. Stability of a product has nothing to do with development and everything to do with what is released (hence why it is always on top). If you want to hide it, better to remove it entirely then. Will revert this in the meanwhile except for a good reason (which I doubt).


In practice, you would probably propagate the exception, otherwise callers of this constructor would never know that something has failed.


// FIXME: See if we can figure out a way to set this to 'AES-256-CBC' for new installs only


@bodil It was not the breaking change (changing `ErrorT`) that caused builds to fail, but rather the fact that our other packages depended on the old version of `transformers`. So, what happened was that it was not possible for `bower` to decide upon a compatible version of `purescript-transformers`; this caused the build to fail even before the breaking change would have been able to cause a problem.

I am not super familiar with the rules for semantic versioning, and it may be the case that dependency bumps are not traditionally meant to result in major version changes, but a change like this (even if there were no code changes) can frequently result in breakage across the ecosystem, I think. At SlamData, for this reason we always increase at least the minor version when dependencies are upgraded by at least `+0.1.0`; this practice allows us to incrementally upgrade our packages without anything breaking in between.

Thanks for releasing the new version! :smile:


Final grade: A
Excellent documentation and implementations, as well as highlighting potential alternatives or compromises. It was a pleasure to review your code!


Not good early code :D.


My mark is 90.


Great!


Please ask questions on the forum if you want them answered.


This kind of documentation is excellent! Either here or in a README.md file, it is great to see such in-depth explanation of the rationale and the approach! Keep up the great work!


Additionally, when mutating existing tags, what do you think the best policy is? My instinct is that even if a tag was bad (like `v2.0.1`), it should not be deleted, since this will also break builds. For instance, I will now have to go through and upgrade my packages again, since they will now fail to acquire a version of `purescript-test-unit` satisfying the bound `^2.0.1` (in the interim, I had already adapted our packages to use this bound).


Hey This should really be 100px tall


No way, this is way better.


Hey, this is cool.


Alternatively, Multimap from Guava could be handy here. http://docs.guava-libraries.googlecode.com/git/javadoc/com/google/common/collect/Multimap.html


You should probably change the wording here too.


:heart:


Well done for caching the list of calculated entries!


Install git and/or alias your git executable to git and/or add git to your PATH.


Final grade: A
Great to see such detailed documentation, and the implementations seem solid too!


Ordinarily, tags should definitely stick around forever, but in this case I figured you were probably the only one using the tag yet, so I went for it.

Ideally, of course, we'd have a release tool which decides on semver bumps based on type signatures, like Elm has, so these things don't need to happen in the first place. Maybe some day...


Both those keys work with the current version of ovr.getFloat(...). I found a post in the forums suggesting that OVR_KEY_IPD is read-only, so ovr.setFloat(...) won't work with that parameter. https://forums.oculus.com/viewtopic.php?t=20369 I'm putting a few calls to ovr.getFloat(...) into the unit tests, to make sure that they at least do not crash.


@bodil OK, fair enough. Not much harm done anyway (just a few frantic releases in our packages, but I don't think anyone was affected).

thanks again!

btw, I wasn't aware that Elm had this feature. That sounds really cool.


It's not testing sass yet, right?


I feel like there is too much mutable state in the class itself, which should instead probably be pushed as much as possible into the methods. This would then allow for instance to potentially run multiple methods in parallel on multiple cores if you had them available.


Oh fuck. Sorry forgot to switch to a branch. It's just a pdf doh T.T mbmbmb @jodreen


@svenstucki check the docs if you have problems with the detail view


This works, but only in the basic version with exactly two words obviously :)


imgur supports https


Note that you could cache this array so that it can be reused to speed up subsequent calls.


Final grade: B+
Implementations seem reasonable and well organized, but in some cases could be generalised a bit more, and the documentation could be improved :)


Yes I know....that's what I said directly.  I am asking ...where is this make file located?  folder?  app?
Thanks


Glad to see just like last time you've already given up


Aye, this was discussed in the PR. @tvandijck said he already had a fix for it among his bag of tricks which he'd submit a PR?


Generic "Tuple" or "Pair" classes are not always a good idea, as they are too abstract and it is hard to reason about how they are used in practice; in this case for instance, you could have called the fields `startIndex` and `maxLen` and it would have been obvious what they mean.


public?


private?


It would be good to document your assumptions about the input data, such as that the dictionary is sorted, given that it is not possible to enforce it with the type system alone.


This is quite hard to read; maybe split the two conditions into separate variable with meaningful names?


Final grade: B
Overall good implementations, but not always the most elegant solutions were chosen. And more documentation would have been appreciated :)


Array index were wrong, should read
score -= Unsupported[weaktype][opposed]


@phonohawk  Does this work for you? It doesn't for me. On my linux box, `-c '%s'` is used with `stat` to get file sizes. See http://man7.org/linux/man-pages/man1/stat.1.html

The other weird thing that was happening is that even though "stat" gave an error the test succeeded. For example

```
if testg `fofdsas` -ne `dfdas`; then
    echo "ok"
else
    echo "Nope"
fi
```
prints "Nope".

Other than this, all of this is really super. Thanks. If I don't hear back in a little while, I'll merge this with the small change to that line of code.


Put this after `NameServers`.


Good catch.


@mbassermann Mybatis-velocity has been released.  It should show up in central in the next couple of hours.  Please let me know if there are any issues with the release.


looking at this and the past code in a split screen makes it look like this is an issue


Then so be it. It has been committed to the codebase :)


Hello! I am looking for a way to contact someone involved in the gvNIX development group to offer my support.  I have prepared some notes and suggestions for the gvNIX 2.0.0.M1 quickstart tutorial that I'd like to share.  I am also working on translating their reference documentation from Spanish to English.
I can't figure out how to send private messages in Git, so please contact me on huffmaki at yahoo.  If not, are you able to pass this to someone who is, please?

My name is Curt Huffman.  I've been in the IT industry for over 25 years and have just started using  Roo and gvNIX.  I am eager to support the open source community, but my coding skills are limited as I am re-learning it all.  (A lot has changed in 25 years!)  That said, I have many other skills that I might be able to contribute.
Cheers.
-Curt Huffman

Hola!
Estoy tratando de ponerse en contacto con alguien en el grupo de desarrollo gvNIX ofrecer mi apoyo. He preparado algunas notas y sugerencias para el gvNIX 2.0.0.M1 Quickstart tutorial que me gustaría compartir. También estoy trabajando en la traducción de la documentación de referencia del español al Inglés.
No puedo encontrar la manera de enviar mensajes privados en Git, así que por favor póngase en contacto conmigo en huffmaki en yahoo.  Si no, ¿eres capaz de pasar esto a alguien que es, por favor?

Mi nombre es Curt Huffman. He estado en la industria de TI de más de 25 años y acabo de empezar a utilizar Roo y gvNIX. Estoy ansioso por apoyar a la comunidad de código abierto, pero mis habilidades de codificación están limitadas ya que estoy re-aprendiendo todo. (Mucho ha cambiado en 25 años!) Dicho esto, tengo muchas otras habilidades que yo podría ser capaz de contribuir.

Por favor, perdona la torpe español. Mis habilidades españoles son mayores de mis seres codificación!  Aprendí mexicana en la escuela secundaria, pero el Google Translate no es malo.

Saludos,
- Curt Huffman


inttype.h is introduced in vs2012. Ether patch the file or move to vs2013
or after?

Qian

On Fri, Oct 2, 2015, 6:51 PM Keith Fischer <notifications@github.com> wrote:

> Which SDKs & Visual Studio version is being used that supports inttypes.h
> ? Were you able to build the entire solution after this commit? I'm trying
> to determine what mixture of toolsets is needed.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/google/sagetv/commit/b90155b885b512d07f950c8a64bc939849e3218b#commitcomment-13573911>
> .
>


note for self:
prepend this with $tmp


Neat. How does it feel? It looks mostly nicer.


note for self:
I think this else needs to be stripped out


thanks for the patches however im getting this error with the latest patch sets

device/asus/mofd-common/libhealthd/healthd_board_mofd_v1.cpp: In function 'int mapChargeRateString(const char*)':
device/asus/mofd-common/libhealthd/healthd_board_mofd_v1.cpp:32:16: error: 'BATTERY_CHARGE_RATE_FAST_CHARGING' is not a member of 'android'
         return android::BATTERY_CHARGE_RATE_FAST_CHARGING;
                ^
device/asus/mofd-common/libhealthd/healthd_board_mofd_v1.cpp:34:16: error: 'BATTERY_CHARGE_RATE_UNKNOWN' is not a member of 'android'
         return android::BATTERY_CHARGE_RATE_UNKNOWN;
                ^
device/asus/mofd-common/libhealthd/healthd_board_mofd_v1.cpp:35:1: error: control reaches end of non-void function [-Werror=return-type]
 }
 ^

any idea on how to fix it?


@BennyDonut,

Ok, looks good :+1:

10/10


commit comment


Thanks for the review! I've modified for the two places but I cannot do the style change. The checkstyle will fail the build.


#5 问题出现在这次commit里


Yes, it should say the longest. If you check the master branch, you should see that I caught and fixed this mistake a short while after making it.


More commit comments


what's the idea of having this `.exists` statement here?


Apologies about that, for some reason it doesn't work for me if I don't use that though! It's really annoying and I have no idea why it does that :(

P.S. Yes I generated a new world but it still won't work.


something


@MrGrier5,

* good usage of the `puts` method and the `print` method
* good variable names
* remember, I found out you don't have to use the `chomp` method if you're using the `to_i` method because it already strips the extra line from the string
* good job putting spaces in between your methods and your parameters

10/10


10/10

In the future:

* when assigning variables, put spaces on each side of the assignment operator (=)


hi


this should really be returning `a` after the loop, yeah?


@Tamrioda (good user name! :)

10/10, but next time:

* add a space between your methods and your parameters if you're not using parenthesis
* when assigning variables, put a space on each side of the assignment operator
* i found out `.chomp` is not needed when using `.to_i` since it already strips the extra line


@david-truong can we break at least after the first "," Or put it all in one line?

It's hard to know that the {} is for result[1] and not for result[0]


@jeremycardine good job! 10/10


These are good


can we just declare this function as a private method on the element instead of inside of another function?


i'll mess around and see what works


what are these new variables for?


As are object not getting pre-casting benefits from having these references. Maybe have one bit flag that is ```_featureOverrriden``` and set to 1 if is true; and put the overriden type in ```MaybeExtra``` if flag is ```0``` cast ```this``` if  ```1``` return  ```MaybeExtra?.TryGetValue(key, out feature)```.

For ```FastReset``` then just set ```_featureOverrride = 0```


/s/memory usage/memory allocation/


Title: Math Fall/ タイトル: マスフォール
A random number fall from the ceiling.
You can control lateral direction by cursor keys.
The row or the column is deleted when sum of row or column equals target number.
If the random number reached the ceiling, this game is over.
Turn off 'Hint; sum of  each row or column', if you want to play  this game more difficultly.

ランダムの数字が上から落ちてきます。
カーソルで左右に動かせます。
右側・下側にあるターゲットナンバーと縦の列もしくは横の行の数字の合計が一致するとその行もしくは列が消えます。
数字が上まで積み上がったらゲームオーバーになります。
ヒントのチェックボックスをオフにするとヒント（合計値）が表示されなくなり、難易度が上がります。


See http://www.alternativeintersections.org/intersections/view/507 for the new intersection.


How about fixing this, too?


Fixes #7


You should also make MDC2_CTX.num a size_t.


6/10 points

* when the computer runs this program how will it know to use Ruby?
* where do you tell the user how much they would make?


10/10 :+1:


@shemaiahmccall good job, 9/10

* make sure not to put spaces in the file name :)
* when assigning variables, put a space on each side of the assignment operator (the equals sign)


@jython234 Alright, I'll add a getUsername() soon. But getName() is for all entities in general, nametags are for their display name; not internal handling.


m_uiPhase = 0 missing.


assignment 5


Before the comment-out, the crash handler always failed to read the version number and output a strange value (such as 262159 on my Windows XP, or 0 on my Windows 7). I still believe there is a clean solution.


10/10 :+1:

in the future, put spaces on each side of your assignment operator when assigning variables :) Also when using the .to_i method, you do not need .chomp


What happens when you run this file? Does it work successfully?


Thanks Don. I knew this was a problem from the beginning and forgot all about fixing it. It's nice having someone recognize where I goofed.


...to be honest, I think I forgot to uncomment that out. However, I can at least explain a little bit of what happened with regards to mapconv.

I'm no longer using the version updater program. Likewise, I got rid of using `version_num`. All uses were replaced with `sm_version_git_hash` or the number 0, depending on purpose.

I can confirm that I at least got a stack trace at this point. There were even some english looking methods for parts of it. However, I suspect more work will need to be put in.

Again, I am struggling a bit with this. If you can improve it, go right ahead.


Very good! 10/10


how to test?


when using conditionals, make sure to indent the lines like so:
````
if total >= 1000
  stotal = total.to_s
  ctotal = stotal.insert 1, ","
  puts "you will earn #{ctotal}"
else
  print "your income would make #{total}"
end


@LizzTheLizard what would happen if we had a total of more than 9,999?


Ah chales acabo de ver eso, pero si lo voy a hacer ahora que tenga un chance, gracias por el espacio @domix


It doesn't work i realized that when i ran it in cloud 9 i changed it on there and must have forgot to change the code on git hub


Is anyone able to place a note on https://sourceforge.net/projects/bopm/ pointing here and to hopm? I think that could reduce some confusion.

Thanks!


10/10 :+1:


Inline code blocks can't have links. Do you think it would be better if I removed the code ticks?


what about if the total is 1,000? :)


제가 이렇게해놓은 이유가 json에서 네임 이름으로 파일을 만드는데 파일 이름 사이즈가 초과되면 문제가 생겨요.


make sure to indent lines within a conditional


ok!


아무레도 다시 바꾸신다고하면 name 길이를 제안해주셨으면 해요.


This is an important 1.13.x fix.


nice


mostHealthCreature is already defined on line #L670


I guess so. I was just poking around the code, cleaning things up and tweaking them. Before I knew it, I had all these changes!

I'm making note to commit more frequently. I'm also planning to start using the beta branch again, now that my code is slowly approaching stable. If I'm gonna use Git, I might as well get it right :)


r+


Since Go 1.5 failed to build here due to an (it turns out unrelated) test failure, and this touches ~600 Go packages, I was weary to push this to master. Will do next time. Thanks!


Would it be better to grab the trip data outside of the calculateExpense function and then pass it in as an argument?


saw approval from jedmccaleb
at https://github.com/graydon/stellar-core/commit/21855d2c59fb14bbccd82ae69ef1c02ed9e2fba8


merging graydon/stellar-core/archive-publisher-assert = 21855d2c into auto


graydon/stellar-core/archive-publisher-assert = 21855d2c merged ok, testing candidate = bcb13667


``` CPP
commit 08681ef30b0784ea5a8b45b8d363caf49f7b8cd1
Date:   Sat Oct 3 09:40:03 2015 +0600

    Iron Hall Guardian - Magmus encounter - Add script.

diff --git a/scripts/eastern_kingdoms/blackrock_depths/blackrock_depths.cpp b/scripts/eastern_kingdoms/blackrock_depths/blackrock_depths.cpp
index eb23735..5c040a1 100644
--- a/scripts/eastern_kingdoms/blackrock_depths/blackrock_depths.cpp
+++ b/scripts/eastern_kingdoms/blackrock_depths/blackrock_depths.cpp
@@ -1075,6 +1075,105 @@ bool GossipSelect_boss_doomrel(Player* pPlayer, Creature* pCreature, uint32 /*ui
     return true;
 }

+/*######
+## npc_ironhand_guardian
+######*/
+enum
+{
+    // spells
+    SPELL_GOUT_OF_FLAME                 = 15529,
+    SPELL_STONED_VISUAL                 = 15533,
+};
+
+struct npc_ironhand_guardianAI : public ScriptedAI
+{
+    npc_ironhand_guardianAI(Creature* pCreature) : ScriptedAI(pCreature)
+    {
+        m_pInstance = (ScriptedInstance*)pCreature->GetInstanceData();
+        Reset();
+    }
+
+    ScriptedInstance* m_pInstance;
+
+    uint32 m_uiGoutOfFlameTimer;
+    uint32 m_uiAggroModeTimer;
+    uint8 m_uiAggroModeStage;
+
+    void Reset() override
+    {
+        m_uiGoutOfFlameTimer    = 0;
+        m_uiAggroModeTimer      = 0;
+        m_uiAggroModeStage      = 0;
+    }
+
+    void AttackStart(Unit* /*pWho*/) override { }
+    void MoveInLineOfSight(Unit* /*pWho*/) override { }
+
+    void ReceiveAIEvent(AIEventType eventType, Creature* pSender, Unit* /*pInvoker*/, uint32 /*uiMiscValue*/) override
+    {
+        if (!m_pInstance && pSender->GetEntry() != NPC_MAGMUS)
+            return;
+
+        switch (eventType)
+        {
+            case AI_EVENT_CUSTOM_A:
+                m_uiAggroModeTimer = 1000;
+                break;
+            case AI_EVENT_CUSTOM_B:
+                m_uiAggroModeTimer = 0;
+                break;
+            case AI_EVENT_CUSTOM_C:
+                m_uiAggroModeTimer = 0;
+                break;
+        }
+    }
+
+    void UpdateAI(const uint32 uiDiff) override
+    {
+        if (!m_creature->SelectHostileTarget() || !m_creature->getVictim())
+            return;
+
+        if (m_uiAggroModeTimer)
+        {
+            if (m_uiAggroModeTimer <= uiDiff)
+            {
+                switch (m_uiAggroModeStage)
+                {
+                    case 0:
+                    {
+                        m_creature->RemoveAurasDueToSpell(SPELL_STONED);
+                        if (DoCastSpellIfCan(m_creature, SPELL_STONED_VISUAL) == CAST_OK)
+                            m_uiAggroModeTimer = 1000;
+                        break;
+                    }
+                    case 1:
+                    {
+                        m_uiGoutOfFlameTimer = 5000;
+                        m_uiAggroModeTimer = 0;
+                        break;
+                    }
+                }
+                ++m_uiAggroModeStage;
+            }
+            else
+                m_uiAggroModeTimer -= uiDiff;
+        }
+
+        if (m_uiGoutOfFlameTimer < uiDiff)
+        {
+            if (DoCastSpellIfCan(m_creature, SPELL_GOUT_OF_FLAME) == CAST_OK)
+                m_uiGoutOfFlameTimer = 15000;
+        }
+        else
+            m_uiGoutOfFlameTimer -= uiDiff;
+    }
+};
+
+CreatureAI* GetAI_npc_ironhand_guardian(Creature* pCreature)
+{
+    return new npc_ironhand_guardianAI(pCreature);
+}
+
 void AddSC_blackrock_depths()
 {
     Script* pNewScript;
@@ -1139,4 +1238,9 @@ void AddSC_blackrock_depths()
     pNewScript->pGossipHello = &GossipHello_boss_doomrel;
     pNewScript->pGossipSelect = &GossipSelect_boss_doomrel;
     pNewScript->RegisterSelf();
+
+    pNewScript = new Script;
+    pNewScript->Name = "npc_grimstone";
+    pNewScript->GetAI = &GetAI_npc_grimstone;
+    pNewScript->RegisterSelf();
 }
diff --git a/scripts/eastern_kingdoms/blackrock_depths/instance_blackrock_depths.cpp b/scripts/eastern_kingdoms/blackrock_depths/instance_blackrock_depths.cpp
index 5334bec..f8c8f5b 100644
--- a/scripts/eastern_kingdoms/blackrock_depths/instance_blackrock_depths.cpp
+++ b/scripts/eastern_kingdoms/blackrock_depths/instance_blackrock_depths.cpp
@@ -333,6 +333,7 @@ void instance_blackrock_depths::Load(const char* chrIn)
 void instance_blackrock_depths::OnCreatureEnterCombat(Creature* pCreature)
 {
     if (pCreature->GetEntry() == NPC_MAGMUS)
+        SendAIEventAround(AI_EVENT_CUSTOM_A, pCreature, 0, 150.0f);
         SetData(TYPE_IRON_HALL, IN_PROGRESS);
 }

@@ -363,6 +364,7 @@ void instance_blackrock_depths::OnCreatureEvade(Creature* pCreature)
             SetData(TYPE_TOMB_OF_SEVEN, FAIL);
             break;
         case NPC_MAGMUS:
+            SendAIEventAround(AI_EVENT_CUSTOM_B, pCreature, 0, 150.0f);
             SetData(TYPE_IRON_HALL, FAIL);
             break;
     }
@@ -408,6 +410,7 @@ void instance_blackrock_depths::OnCreatureDeath(Creature* pCreature)
             SetData(TYPE_TOMB_OF_SEVEN, DONE);
             break;
         case NPC_MAGMUS:
+            SendAIEventAround(AI_EVENT_CUSTOM_C, pCreature, 0, 150.0f);
             SetData(TYPE_IRON_HALL, DONE);
             break;
     }
```
Try this for comparison.


some tests failed:
failure: https://travis-ci.org/stellar/stellar-core/builds/83418042


Lookin hotter C:


@yeshwanthvshenoy  beech ;-;


Why that? Couldn't we just delete the line?
(and below as well)


Hello can somebody create a release with this change ?
This bug fix as serious impact
Thank you


@latobarita: retry


We all need two spaces after periods here, please.


Good catch!  Thanks for this one!


Awesome, thanks for doing this!


@jon-a-nygaard


I wonder why dict() here...


Playing with the sampling rate.


We all need two spaces after periods, please.


graydon/stellar-core/archive-publisher-assert = 21855d2c merged ok, testing candidate = d9fd2791


Please add this https://github.com/Euphoria-OS-Devices/android_device_lge_g2-common/commit/7498250857ea25130c88d8d1d58508aad5209e91 to fix vocal message on whatsapp and telegram


Я писал, что должен быть не первый-второй семестр, а описатель для вывода всех семестров, так как их может быть до десяти.


I think we need it like this, no?
````csharp
return left?.Equals(right) ?? false;
````


Loading this commit on my VM makes the sidebar invisible even after the page is done loading. :cry:

<img width="1440" alt="screen shot 2015-10-02 at 10 09 53 pm" src="https://cloud.githubusercontent.com/assets/4585431/10261551/61823274-6952-11e5-8720-8e83932670ce.png">


ref #57


This is a git mirror of ddwrt's svn repository


all tests pass:
success: https://travis-ci.org/stellar/stellar-core/builds/83419287


fast-forwarding master to auto = d9fd2791


Ack, this is pretty complex. But, it's pretty good. I think we need this:
````csharp
public StateItemKey(Type type, String key) {
    if (type == null) throw new ArgumentNullException(nameof(type));
    Type = type;
    if (key == null) throw new ArgumentNullException(nameof(key));
    Key = key;
}
public Type Type { get; private set; }
public String Key { get; private set; }
````
Helps push toward the correct implementation. What do you think?


   Date: Fri, 02 Oct 2015 15:49:19 -0700
   From: gregory-marton <notifications@github.com>

   Taylor, if you can remember back this far, do you know why you left in this "if False"?

   If I try to run it now, I get a "syntax error near bayesdb_generator_column" via the update in commit 558674baad707fb80a242e5f5a579884df23af0d

   Do you think this test should get fixed and run?

I assume you mean the `if False:' in test_legacy_models.py?  No, I
don't remember.  Maybe it took too long; maybe it never worked in the
first place but there was a nugget I wanted (and failed) to remind
myself about; maybe it wasn't stable.

The `foo JOIN bar USING (c)' notation can be replaced by commas and a
WHERE clause:

SELECT ... FROM foo, bar WHERE foo.c = bar.c

I never implemented any of the various explicit JOIN notations in BQL.


In case you're still getting these notices: pretty proud of this one: I created the "This day in history" method for shows and kept having issues because the local DB is Sqlite3 and the production one is Postgres. So I did this, and I did NOT find it on Stack Overflow!

```
def self.today_in_pit
    if ENV['RAILS_ENV'] == 'development'    # sqlite3 version
      where("strftime('%m', date) + 0 = ? and strftime('%d', date) + 0 = ?", DateTime.now.month, DateTime.now.day)
    else # postgres version
    where("extract(month from date) = ? and extract(day from date) = ? ", DateTime.now.month, DateTime.now.day)
    end
  end
```

That is all. (actually testing some "eager loading, too").


agreed


>> Helps push toward the correct implementation. What do you think?
perfect.   will use that for Plan B+


Interesting. This is easy enough to read, though.


Exciting!


Please read the comments I made in code before methods, BTW.


shouldn't you add return here?


Может лучше загружать юзеров, когда мы точно знаем, что это список пользователей(проверка ниже) ?


네, 무슨 말씀인지 알겠습니다. 참고할게요.


@markguinn Looks like this has been resolved? Latest build passes?


TeamCity Azmq :: Build (Linux) [Build 138](https://78.46.71.115/viewLog.html?buildId=612&buildTypeId=Azmq_BuildLinux&guest=1) is now running


TeamCity Azmq :: Build (OSX) [Build 121](https://78.46.71.115/viewLog.html?buildId=613&buildTypeId=Azmq_BuildOsx&guest=1) is now running


TeamCity Azmq :: Build (Windows) [Build 168](https://78.46.71.115/viewLog.html?buildId=614&buildTypeId=Azmq_Build&guest=1) is now running


TeamCity Azmq :: Build (Linux) [Build 138](https://78.46.71.115/viewLog.html?buildId=612&buildTypeId=Azmq_BuildLinux&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 28 Build time: 00:00:14


TeamCity Azmq :: Build (Windows) [Build 168](https://78.46.71.115/viewLog.html?buildId=614&buildTypeId=Azmq_Build&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 28 Build time: 00:00:22


TeamCity Azmq :: Build (OSX) [Build 121](https://78.46.71.115/viewLog.html?buildId=613&buildTypeId=Azmq_BuildOsx&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 28 Build time: 00:00:22


@Rocky640 ok, I completely missed that this was only scoring endgame. Thanks for pointing out.
@snicolet Congrats for this idea!


Congratulations!


@devhell fixed in 5d952d68deadca348d611e2d6fcda2c893f50b2a


Согласен с Максом по поводу того, что у нас много логики в list. Нужно доопределить блок list соответствующими методами в `list_type_channels.js` и `list_type_users.js`


getline is not a GNU extension. It is a POSIX function.


:frowning: Teo's OCD demands constancy! (not jokin, making things the same is my version of counting random crap)
https://github.com/DarkstarProject/darkstar/commit/10af605ba71a95686c7377d40166d7d88bdeb656
https://github.com/DarkstarProject/darkstar/commit/712c753aa84b4a733a0790673141ee7f8c3834dc

And luckily for me, I wasn't the only one to find the missing space between if and left parenthesis hard on the eyes, so when demo made those all the same they all got a space. I have to zoom in pretty far to read some of these things, bad eyes and crappy monitor.


Когда будешь адаптировать под новую версию, название метода заодно поменяешь


I suppose you could, however the button logic and the constant switching between LF & HF images...

There is no gold tool,  if you want to have a pentesting tool searching and cloning tags on the fly, I suggest you buy 2 pm3's and hook them up with different modes in a case...   or something like that.


This python script creates a new trigger tower mapping file that more closely resembles the CMSSW tracker geometry.

The old mapping file has these number of modules:
  Tower 0 has 489 modules.
  Tower 1 has 486 modules.
  Tower 2 has 486 modules.
  Tower 3 has 489 modules.
  Tower 4 has 489 modules.
  Tower 5 has 485 modules.
  Tower 6 has 485 modules.
  Tower 7 has 489 modules.
  Tower 8 has 534 modules.
  Tower 9 has 520 modules.
  Tower 10 has 520 modules.
  Tower 11 has 534 modules.
  Tower 12 has 534 modules.
  Tower 13 has 516 modules.
  Tower 14 has 516 modules.
  Tower 15 has 534 modules.
  Tower 16 has 414 modules.
  Tower 17 has 410 modules.
  Tower 18 has 410 modules.
  Tower 19 has 414 modules.
  Tower 20 has 414 modules.
  Tower 21 has 402 modules.
  Tower 22 has 402 modules.
  Tower 23 has 414 modules.
  Tower 24 has 415 modules.
  Tower 25 has 412 modules.
  Tower 26 has 412 modules.
  Tower 27 has 415 modules.
  Tower 28 has 415 modules.
  Tower 29 has 403 modules.
  Tower 30 has 403 modules.
  Tower 31 has 415 modules.
  Tower 32 has 539 modules.
  Tower 33 has 525 modules.
  Tower 34 has 525 modules.
  Tower 35 has 539 modules.
  Tower 36 has 539 modules.
  Tower 37 has 521 modules.
  Tower 38 has 521 modules.
  Tower 39 has 539 modules.
  Tower 40 has 486 modules.
  Tower 41 has 484 modules.
  Tower 42 has 484 modules.
  Tower 43 has 486 modules.
  Tower 44 has 486 modules.
  Tower 45 has 482 modules.
  Tower 46 has 482 modules.
  Tower 47 has 486 modules.

The new mapping file has these number of modules:
  Tower 0 has 467 modules.
  Tower 1 has 467 modules.
  Tower 2 has 467 modules.
  Tower 3 has 467 modules.
  Tower 4 has 466 modules.
  Tower 5 has 466 modules.
  Tower 6 has 466 modules.
  Tower 7 has 466 modules.
  Tower 8 has 510 modules.
  Tower 9 has 515 modules.
  Tower 10 has 515 modules.
  Tower 11 has 510 modules.
  Tower 12 has 511 modules.
  Tower 13 has 516 modules.
  Tower 14 has 516 modules.
  Tower 15 has 511 modules.
  Tower 16 has 391 modules.
  Tower 17 has 400 modules.
  Tower 18 has 400 modules.
  Tower 19 has 391 modules.
  Tower 20 has 392 modules.
  Tower 21 has 401 modules.
  Tower 22 has 401 modules.
  Tower 23 has 392 modules.
  Tower 24 has 403 modules.
  Tower 25 has 412 modules.
  Tower 26 has 412 modules.
  Tower 27 has 403 modules.
  Tower 28 has 403 modules.
  Tower 29 has 412 modules.
  Tower 30 has 412 modules.
  Tower 31 has 403 modules.
  Tower 32 has 520 modules.
  Tower 33 has 525 modules.
  Tower 34 has 525 modules.
  Tower 35 has 520 modules.
  Tower 36 has 521 modules.
  Tower 37 has 527 modules.
  Tower 38 has 527 modules.
  Tower 39 has 521 modules.
  Tower 40 has 462 modules.
  Tower 41 has 462 modules.
  Tower 42 has 462 modules.
  Tower 43 has 462 modules.
  Tower 44 has 461 modules.
  Tower 45 has 461 modules.
  Tower 46 has 461 modules.
  Tower 47 has 461 modules.


OK


Add `check_csrf` in `moderate.php` for `action=all` or is this one necessary?


Please remember the deadline is 5pm each Thursday.
I will give you penalty next time if you commit later than that.


@edumntg tive este problemas ao compilar:
![sem titulo](https://cloud.githubusercontent.com/assets/13180782/10261697/1487b222-697b-11e5-8980-4da4f1476b0f.png)
I how this problem when compiler

Pode ajudar?
Can you help me?


yes, I greped it already again: https://github.com/Clooney82/MavLink_FrSkySPort/commit/bfaece2de9b0d4248a7b542ef3921b50a2d25ef3


Oops, corrected in code posted to hills: 'VMD'


NOOOOO!!!! THE BADGES!!!!!


meme is good


Does this make sense?
```dic[j] != 0 && dic[j] % 2 == 1```


Wrong description?


please take look


yeah, my bad


Use StringBuilder instead of String in a loop.


that's what i get for trying to document my code


wrong


Co minutę
Co 5 minut
Co 15 minut
Co 30 minut
Co godzinę


Miss follow up problem.


if only i knew korean


Wow, gradients, animations!
Something weird with the model though, when I run in the browser it is not learning. Just one timestep every 20ish steps has any predictions. Even after 1000 time steps.


`month` and `next_month_tick` are now unused, remove them


very good.


Just use `com.badlogic.gdx.utils.Disposable`, don't roll your own interface


Seems to me this commit belongs in " mixed-mode: add netfilter runmode "


Don't name it Label, add a prefix or something. Also, use composition instead of inheritance. We don't want to expose all the methods from Scene2D label. Turn the label into a field.


Same thing here, don't extend the class you're wrapping.


If you're testing performance you might as well use a prepared statement here, you're generating a string query everytime, a prepared statement would just send an id to the server and a bunch of values, much more efficient.


Thats really odd. its using *.dll as the search, and shouldn't be loading anything with dlln extension, and doesn't on mine, or the build server. But this is probably good to have just in case. That's why I went with .dlln in the first place, because it was just causing lots of problems anyways.


Do not merge yet I still need to verify the examples are still working.


Shoutout to ShiroSuigintou KappaHD


`luaotfload-tool` is not necessary since we are doing a fresh installation and there is no cache.

BTW, I find `luaotfload-tool` is broken in the latest TeX Live. It is dut to lack of `luatexbase.loader.lua`. This file has been removed by package `luatexbase`. But `luaotfload-tool` makes an assertion for it.


That was my thought too. It was really weird.


@mikermcneil this one? https://github.com/balderdashy/sails-generate-new/blob/master/templates/app.js

I don't see it.


Do we have any idea when or if they will be releasing a non-prerelease version of the Experimental Collections package? It's causing the NuGet packing to fail. I'm going to try and see if there is a workaround, but it does not like building a stable package with an unstable dependency.


Yes, but it's really heavy to know why :+1:


- Doh, overlooked this.... in the end (since updated), I had changed the 'values' to small abbreviations to use as keys for the values returned. This can be more clearly seen how the associative array now functions with the topics boxes.
- For just a single space,   requires so much less than coding a class or messing with goofy inline padding and such. Think of it as using a kitchen knife to cut an apple instead of a chainsaw.
- granted this is a late reply, those topics boxes are also now being generated via a loop


@qinglee  There's only a fresh install if we are not using a Travis cache, and the aim of editing the script was to allow for that whilst still having the deps change (and not have me need to manually clear the Travis cache).

I'll look at the `luatexbase` business: that's annoying (asserting for internal organisation of another package is daft). We checked all of the (La)TeX stuff, but that one didn't show up. In any case, I can probably make the cache reset a bit easier: just delete the directory.


@Zhejiang = @浙江 ...


I don't think there is a workaround. I had forgotten about the nuget packaging.  ill swap it out for an collection that will build.


What benefit do we gain by ignoring the peer variable and forcing the FQDN to be used? It's quite confusing when your servers start up and do not create a volume even when your peer attributes appear to be correct. Why did you choose to go for the fqdn override?


Can you explain a bit more in the commit message why this is necessary?


Work in progress it will be fixed.


```return odd_occurrence <= 0```


Good catch!


Hello, thank for fixing the bugs :). However, I can still see people putting more than 1 flag. I played after the update.So can you check it out please. Thanks. Keep up the good work.


OK, but not the best method.


This is a really cool program that does just about everything you'd want with regards to a garage door.  Wonder if you are still working on it?   There is the below error for the line
runonce(now + closeAfter, closeWhenDark)

I'm just a beginner with groovy but I don't see 'now' being initialized in the program.

groovy.lang.MissingMethodException: No signature of method: script14438522744751066302953.runonce() is applicable for argument types: (java.lang.String, java.lang.String) values: [now3, closeWhenDark]
Possible solutions: runOnce(java.lang.String, java.lang.Object), runOnce(java.util.Date, java.lang.Object), runOnce(java.lang.String, java.lang.Object, java.util.Map), runOnce(java.util.Date, java.lang.Object, java.util.Map), unwire(), run() @ line 193


Looks good.


Maybe specify the capacity based on the number of pairs being added, and add a `Debug.Assert` to ensure the capacity and count match in a DEBUG-only static ctor? Along the lines of: https://github.com/dotnet/corefx/commit/425d8e7979fbefb7cdb5f44f557932ff5a9ae570


ok


Please avoid using !important.  Using !important is only necessary when someone else stated !important in a rule with insufficient selectors to make it relevant.  Using !important is good for a quick solution in experimental code, but it does more damage than good in production code.   It forces the next coders to state !important when they write their rules and eventually !important becomes meaningless.  The relevance of a rule can always be set by using (enough) selectors, unless !important was included somewhere.  Avoid that trap.
Yet if !important was stated in a rule that selects your target, it becomes unavoidable.  But in such case your rule should be specific enough so that it does not propagate the problem started by someone before you.


should return the shorter one


thanks a lot! I've been merged it to the master and will public it in the next update.


Posting from MangoApps to github


Back from github


Why isn't the parameter just Actor then..?


Writing a comment from github


这里后来又改了，语法错了，应该是{0x00}


sdgsdgsdg


<!DOCTYPE html>

    <meta charset="utf-8">
    <title>William Yore III | Designer</title>
    <link rel="stylesheet" href="css/normalize.css">
    <link href='https://fonts.googleapis.com/css?family=Changa+One|Open+Sans:400italic,700italic,400,700,800' rel='stylesheet' type='text/css'>
    <link rel="stylesheet" href="css/main.css">
    <link rel="stylesheet" href="css/responsive.css">
    <meta name="viewport" content="width=device-width, initial-scale=1.0">

JStone Cards

Kimono Series

Portfolio

About

Contact

Front (in Japanese, back in English) ~ My barn having burned to the ground, I can now                see the moon. Inside~Blank.

Front (in Japanese, back in English) ~ I send you these special blossoms, each flower is               a miracle. Inside~Blank.

Front (in Japanese, back in English) ~Imagination is irresistible. Inside~Blank.

Front (in Japanese, back in English) ~ Contrast is the essence of vision.Inside~Blank.

Front (in Japanese, back in English) ~ Fish ghosts swimming in silk stream.                          Inside~Blank.

© 2014 William Yore III.


Don't use `this`, that's what the m prefix is for.


Commenting from github


What do you think of this @whyrusleeping ? I had no triggers that relied on being consumed like that, and it's nice to have the triggers execute in goroutines, in case one takes long.


@divneet please detail problems that you've encountered.


One more comment


Commenting from mA


========== Build: 12 succeeded, 0 failed, 0 up-to-date, 0 skipped ==========


It's usually a good idea to have one expectation / spec. It isolates the failure better.


Do not need these extra spaces.


Ay jios mio que era harto!!!!!