Didn't notice that, great :D I'll add the "double tap" to the translation strings.
Will try your navbar soon :dancers:


TAB + 2 spaces :ok_hand:


Shouldn't that be relative link, like `../foo`? Cause nobody except you have '/Users/mitchellh' :)


`_associated_objs` is not necessarily thread-safe. in general, simultaneous read and write ops should be protected using lock. in other words, we can not read it while updating it.

fwiw, and double checked locking does apply here. as the insert op to `_associated_objs` is not atomic.


It doesn't matter, the test doesn't test the contents, only the existence of the path.


I will check if this is backward compatible and merge as soon as possible.

Thanks!


I will run it again and see if there is a other message to. The funny thing is, that the version.h was in the folder.


yeah, sorry :cry:


Please highlight `username`.


Please highlight `--all` using `:option:`.


I was rather thinking of comparing with the same coefficients but with the wrong formula for computing the probas.


This is not sufficient.  You used a 5 digit guard number on a four digit alignment.  That is safe from the perspective that a `savenum` of `10000` is not truncated back to `0000` since the alignment is a minimum alignment, not a maximum alignment:

```c++
#include <cstdio>

int main()
{
	printf("%02d.pcx\n", 1);
	printf("%02d.pcx\n", 99);
	printf("%02d.pcx\n", 100);
	return 0;
}
```

```
$ ./a.out
01.pcx
99.pcx
100.pcx
```

However, if savenum ever became greater than `9999` (4 nines, not 5), it would overflow the unchecked buffer `savename`, since `savename` is defined as `13 + sizeof(SCRNS_DIR)`.  Spend 4 bytes for `scrn`, 4 more for `.pcx`, 1 for the null, leaving `13 - 4 - 4 - 1 = 4` bytes for the number.

Using 13 bytes for filename made sense when dealing with filesystems limited to 8.3 filenames, but there is no reason to use such a small buffer for `savename` on modern systems, so I am inclined to make it larger, then adjust the guard accordingly.


This returns a Promise now.


Semicolons please :)


This may be susceptible to a race condition. I'd recommend doing this once in init (which automatically gets called package initialization):

```go
var huf HuffmanTree
func init() {
  huf = newFieldpathHuffman
}
```


Stefano,

Once you start committing code, please also add your name to [AUTHORS.txt](ttps://github.com/ahay/src/blob/master/AUTHORS.txt)


Please leave the trailing comma.


Probably not. But the problem is worse with lightness. For instance, there is no such thing as *dark yellow*.


Ok, I'm working on visualizing the RSF data on web browser, could you be interested in?

Thanks for merge!


I'm probably going to do this crap A LOT. If I do it more than once per pull I usually squash before making the pull request. Just FYI.


This file isn't present in the repo, so Node throws on this line :/


@ashwinr64 have a look at this, hope it's ok
I have tested it and it really saves space on the device.
It have no visible effect on performance, try it


$params[] = array(':country', $system->cleanvars($_POST['delete'][$i], 'str'));


I do not agree for this one. Migrations are part of the project...


#6 done


Seems specific no?


You are getting it a second time, why not cache it.


@jglick @oleg-nenashev why not `@CheckForNull` ?


This actually looks like something that belongs to `User` manager.


Is it not automatically added from the urls.py file ?


Did you want i review it to use Yii alias ? I already move from require to Yii import with alias. It work good (and maybe more robust for future release ;) )


Me and Ryan (@byteandahalf) are Retro.
  #GitBash


Ref #1

All is fine, seems to be an issue with haxelib not honoring `classPath`.


Ref #1

All is fine again.


The cores that were fixed are related to the cold war era.


Use a different username for this test user.


Good to know! Thank you! :+1:


Hello cooler can you please make a repack from this source please ?
Would be amazing!


Is it already compilable?


Push it to gerrit.I'll have it merged


As this is a feature spec, try to come up with a way that you could tell from the HTML page if a user is logged in or not. Often this will be checking for the presence (or not) of a message like "Hello someuser" text.

For example, in the foundation, there was a "Hello user@example.com" message shown at the top of each page if a user was logged in.


So we should also add migrations in the repos and MANIFEST.in of all submodules.


I think it is preferable to create a new user fixture instead of reusing `member`.


Слишком длинные строки... имеет смысл обернуть еще в один метод с каким-то названием


I added two more commits that fix 3 of the 4 fails, but the remaining fail does return a slightly different hash.


From memory:
Migrations are already presents in submodules.
And they do not need to be added to the MANIFEST.in file as they are python files (as they are added by default when installing with setup.py).


@yaronct - Oh I see. That complicates things a bit. For instance, functions marked dllimport don't work properly in a static build (and boy does MinGW throw a lot of dllimport errors trying to compile with your cmake). I'm not sure how to handle that building just once for both.


`GetPnPEntityObjectByPnPDeviceId` 是否应为 ` GetPnpEntityObjectByPnpDeviceId ` ？


Much easier!


fixed. Factorio is still yelling to me about an } missing tho >.<


This might be a mistaken addition.


Could you remove the underscore in both of the names, we usually don't name our methods that way.

I also think we could move both declarations outside the `module.exports` block so the function is a bit easier to read.


"Une personne" is more appropriate than "1 personne".


Which one is this for? Is this for the issue with the casting?
If it is, I can fix this pretty soon so it uses the most efficient datatype.


ㅋㅋㅋ코드 리뷰 감사합니다! 이거 토대로 약간씩 수정해볼게요, 한 주 동안 정말 수고 많으셨어요~


`$DEBUG` was deleted? Guessing you reinstate it in a later commit :)


When migrating from South to the new system I removed the "migration"
directories from the git repos and from the virtualenv.

So recreating these directories with the __init__.py file and the 0001
initial (non South) migration  should be done ? or with the migrations
from the virtualenv in the prod server ?

Le dimanche 30 août 2015 à 08:51 -0700, Gabriel Pichot a écrit :
> From memory:
> Migrations are already presents in submodules.
> And they do not need to be added to the MANIFEST.in file as they are
> python files (as they are added by default when installing with
> setup.py).
>
> —
> Reply to this email directly or view it on GitHub.
>
>


Hey, unlike live TF2 you don't need to shell out for the killstreak and then only have it work on one weapon.
This system is a kajillon times better just for that reason.
Good work x3


Oh I see, it's not going to be needed at all with lofg4perl


What are you talking about are the migrations for the rezo app but in this repository we also have migrations for other applications.


I commented out the style because it was being overridden by jQuery UI. We can come back to this later.


@sfan5 i agree with the second part... compiler does their work, debug builds don't need perf improvements...


@MaxXor Could this happen to other files or directories? Perhaps we could create an extension for this (and an equivalent for directories). That would help remove some other random problems that some users might have when dealing with writing to files/directories...


tirei alguns logs e adicionei comentário para a função POST que fizemos exclusivamente para clietes que mandam o POST via o body.


That should only be on Windows, but I will change it to add a NULL termination character just to be safe.


Just so you know -- you pasted the URL twice into the href property.


I don't see them in the GitHub GUI however…

I was also talking about django-paiji2-* (excluding
django-paiji2-social) repos which don’t either have
migrations/__init__.py and migrations/00*.py files in their master
branches.

Le dimanche 30 août 2015 à 09:01 -0700, Gabriel Pichot a écrit :
> What are you talking about are the migrations for the rezo app but in
> this repository we also have migrations for other applications.
>
> —
> Reply to this email directly or view it on GitHub.
>
>


おお、本当だ。何か変な言い回しだと思ってたんですよね…。


Now it works...


This should actually be Router chapter: Commit 5-3


Look good.


Is this why the camera can't shoot with HDR now?


Ben, on cache la partie association ? :)


"wasn't even originally **~~was~~** Perseus"


Magic :(


Unhandled Fundamental Failure


#4942-VRF-with-water-cooled-condenser-does-not-pass-hard-flow-rates-to-plant-loop (rraustad) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1281 of 1281 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg) ![Coverage Badge](http://img.shields.io/badge/coverage%20status-69.99%25-green.svg)


That's a legacy "bug" which I haven't gotten around to changing, yet. In 1.3 it'll be changed to accept a `&[&str]`. It's a "breaking" change, so I at least want to wait until the next minor release.


(Y) nice work


Assignments


Great work on trinket serial!
Is it possible to use PORTB4 instead of PORTB0?


Use custom errors


**** Hello World************

we got ourself the website congratulation 3 stoogies.


Nice, way better than putting this in the html


maybe a function like `addEvent('BG Check')` that would set both the val and name since they are the same


not all are the same. that's why this ...


@tomtor
The reason I did bonus-=depth. Because it is between (average) d^2 and (d-1)^2.
so if d=10
d^2=100
(d-1)=81
adj bonus= d^2-d=90

If the patch passes/fails yellow... I planed to use bonus-=bonus/2.


how to do that? please help @ashwinr64


где модификаторы доступа?


почему паблик?


и туда же, почему линкед хаш мап?


невнятные переменные с именами а и б... это атата


почти все цифры, особенно в таком количестве должны быть константами


может тут ошибка? почему пустая строка?


Because otherwise the lines wrap, so the newline becomes a space. I suppose the intention is to render in 3 lines. :)


цифры и имена переменных


Es fehlt noch das Verbuchen der Vorkasse für PRE-Kunden. Müsste beim "Vertrag unterschrieben" rein (Zeile 198).


тут уместно использовать \t  и \n


Эммм... Серег, это ведь не настоящий твой пароль?


ทดลอง


huh, for some reason I thought NULL here was "I didn't find a new address" rather than failure, but now that I look at the code again I have no clue why I thought that.


Я додав команди для зручності компіляції книги локально у потрібний формат. Вливати ці зміни у головну вітку чи ні - це на Ваш розсуд.
Стандартна команда build генерує книгу в усіх форматах. Мої команди дозволяють згенерувати тільки у потрібному форматі, на стандартну команду це не впливає. Я фактично розбив підкоманди згруповані командою build на окремі групи, що стосуються конкретного формату.
Я раніше також ніколи не писав на Ruby, але мої зміни не містять складної логіки і є просто логічним наслідком з уже існуючого коду.


Looking great! Did you release your inner webdesigner? :)


Congratulations !


#4942-VRF-with-water-cooled-condenser-does-not-pass-hard-flow-rates-to-plant-loop (rraustad) - x86_64-MacOS-10.9-clang: OK (1808 of 1808 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg)


UNET Part *12 - Zombie (Bug Fix)


Warum hast du eigentlich "Flug beendet" und "Rechnung erstellen" als Knöpfe? Diese könnte man doch kombinieren!
Wenn der Flug durchgeführt wurde, gibt's 'ne Rechnung.


#4942-VRF-with-water-cooled-condenser-does-not-pass-hard-flow-rates-to-plant-loop (rraustad) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg)


Das habe ich wegen der kundengruppe gemacht. Rechnung erstellen is immer dann verfügbar wenn PRE und Auftragsstatus=durchführung (also vor beendung) ODER kundengruppe ungleich PRE und Auftragsstatus = beendet


int main (){

    int a = 1;
    int b = 0;
    int c;
    int n = 10; // "input" is 10

    if (n == 0 || n == 1) { // 0 and 1 case
        cout << n << end;
    } else {
        for (int i = 2; i <= n; ++i) { // here you want to reach n
            c = a + b;
            b = a;
            a = c;
        }
        cout << c << end;
    }

    return 0


1. Write a program that prints a fibonaci series that is a sequence of numbers like0 1 1 2 3 5 8.You can vary the number of elements to be printed meaning you can print 10 numbers or 15 or 20 or any desired number.</br>int main (){

    int a = 1;
    int b = 0;
    int c;
    int n = 10; // "input" is 10

    if (n == 0 || n == 1) { // 0 and 1 case
        cout << n << endl;
    } else {
        for (int i = 2; i <= n; ++i) { // here you want to reach n
            c = a + b;
            b = a;
            a = c;
        }
        cout << c << endl;
    }

    return 0


This should be checked in `leave_game`.


Thank you, thank you, thank you so much :+1:


hm...die Rechnung kann doch aber automatisch angezeigt und gebucht werden.

Für PRE und Vorkasse --> "Vertrag unterschrieben"
Wenn durchgeführt --> "Flug beendet"


Write a program print "ping" if a number is divisible by 3,"pong" if a number is divisible by 5, and "ping pong" if number is divisible by both, else print the number.

 (var x=1; x <= 100; x++){
    if( x % 3 && x % 5 ) {
        document.write(x);
    } else {
        if( x % 3 == 0 ) {
            document.write("ping");
        }
        if( x % 5 == 0 ) {
            document.write("pong");
        }
    }
    document.write('
'); //line breaks to enhance output readability
}


Listo, quite lo de los 32 bits y corregí ortografía.


I think that this line hasn't beed added.


Oh, yeah that makes sense. It's just a test network file so I'll just remove it all together.


Sí, estaba ambiguo mi primer ejemplo y repetía algunas cosas más abajo. Intenté simplificarlo e incluir una liga al tip que me diste.


there is a small issue here: I only set lineSmooth and pointSmooth and set their hinting to Nicest. You activate pointSmooth but hint for PolygonSmooth


Write a program  to calculate factorial of a given number. You need to compute the factorial with one logic that uses recursion and another logic without recursion.</br>* Source code to find factorial of a number. */

int factorial(int n);
int main()
{
    int n;
    printf("Enter an positive integer: ");
    scanf("%d",&n);
    printf("Factorial of %d = %ld", n, factorial(n));
    return 0;
}
int factorial(int n)
{
    if(n!=1)
     return n*factorial(n-1);
}


Would it not be better to abstract this to testable methods?

for instance

```php
$this->rootObjects = $rootObjects; // probably best to assign the value when it gets set

$filteredIncludes = array_filter($includedData['included'], [$this, 'filterIncludedData']);
```

```php
protected function filterIncludedData($inclusion)
{
    foreach($this->rootObjects as $rootObject) {
        if ($this->equalResourceObjects($inclusion, $rootObject)) {
            return false;
        }
    }
    return true;
}
```


I really appreciate the work on es dsl, it is powering our search function and making it much easier to deal with es instead of manually writing the match/filter queries. I just found it funny how I was deploying just right when the version bumping took place.


Accidentally added documentation.

There will be a section in the readme describing how to generate custom documentation using jsdoc...

To use autocomplete:

It is the same as bash, where if there is only one option: e.g. ^r[tab] it will use the only name that starts with "r".

Autocomplete is case insensitive. If there are no options nothing happens. On an "empty" tab: ^[tab], all names are printed to the user interface.

If there is more than one option, after printing all available options the user can use this format:

e.g. ^0 r[tab]

to select the 0th option (1st option) of the available names.

Randomguy_ Random Randoms

^1 r[tab] returns: Random to the input box.


maybe have an optional 2nd param, to highlight the strange ones, think it's just Site Change


So urls.py does not seem to be loaded yet, because without this line the left sidebar of Sigefroid does not have all the blocks.


This isn't quite right; the original code included `box move |load_data: LoadData` which was lost during the rebase. I'd prefer restoring that to this solution :)


I'm using the array on 2 more places ..... thinking this is more clear what we do. but you can change it later if you think


Where did you get this from? Don't get me started on copyright and authorship issues again


Congrats! First step is a big step :)


Almost positive this introduced a horrible bug.


Ah looks logical now :-)

Fingers crossed, hope it goes well.


Sorry, I don't think I have JSHint set up in a pre-commit hook on my machine - will need to be more careful..


Ich habe jetzt mal deine Variante durchgespielt. Dort wird mit "Vertrag unterschrieben" der Status auf "Durchführung" geändert und die Knöpfe "Rechnung erstellen" und "Flug beendet" angezeigt. Drückt man dann auf "Rechnung erstellen" wird der KOMPLETTE Betrag in Rechnung gestellt, der Status geht auf "erstellt" und der Knopf "Flug beendet" ist nicht mehr verfügbar.


#4942-VRF-with-water-cooled-condenser-does-not-pass-hard-flow-rates-to-plant-loop (rraustad) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1814 of 1814 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg)


create a migration with

```
change_column :users, :delinquent, :boolean, default: false
```


Hey I dont see the formula to estimate the learning rate: O(Const/sqrt(T+1))?


連続ふぁぼのクソユーザー


先頭にスペースが入っていたので


Fix seems to be effective BTW :)


#4942-VRF-with-water-cooled-condenser-does-not-pass-hard-flow-rates-to-plant-loop (rraustad) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (760 of 760 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg) ![Coverage Badge](http://img.shields.io/badge/coverage%20status-10.24%25-green.svg)


also consider http://docs.seattlerb.org/vlad/  as url


https://ru.wikipedia.org/wiki/Внедрение_зависимости :)


looks good to me


Oh thanks – and good points for `extend self`. :)


Yeah, I know. Problem is I don't know how to use configurate with JSON, so I just did it myself.


@JoachimRohde What is your experience with the merging now since the jdk-1.x-parent branches are gone?


Nope, this doesn't fix it <.<


code modified.


Already taken care of: https://github.com/quasar/QuasarRAT/blob/master/Client/Core/Helper/FileHelper.cs#L96


...Material Independence?


You can write unit tests for `FixedWidthValuesFieldsSplitter`, why didn't you?
`ParseLineTestOneLine` (and your unit tests in general) violates the DRY principle.
In addition, I can't see myself writing (and reading) over and over and over, property assertions like you are doing.
And that disorder in the using section...take a look at how I prefer organized: sorted, by groups, etc.
And remember Robert C. Martin and what he said:
> It is not enough for the code to work.

Please restore the master branch and work in a differente branch like you did in the past (why you didn't this time?) and then, we'll merge...


Oh good. Nice one. :)


Reminder(watchop;shedinja,samurott,combusken,mismagius)


I never suggested it, but yes:

* making `$multiple = true` by default
* removing the `$multiple` parameter

is ok by me.


Why installed ?


options). are loaded
> Hey Dot. What are you doing here ?


@Nephatrine: Ok, let me try and look into these problems.

Btw when I checked out your fork I found that the code u added uses Windows style line endings, which creates a file that mixes Unix style and Windows style line endings. I think it would be best if we stuck with Unix style line endings.


Нет конечно)) да и почта неособо важная))


if **you** find, no ?


Doesn't work anymore on mac.. M I missing something?


This change was only transferred for testing purposes and closed without merging.


This recursion graphic is better than the ones I've seen in textbooks and tutorials.


I like the new search. It looks like it fits perfectly in there.


@yaronct I definitely agree. I guess that's an issue when using multiple tools to write the code that I didn't think about. I'll clean that up.


I do not think this is the right approach, as unless we get CI on these builds this list will get out of date.


Hi @michaelrkn,

Are there plans to highlight the happy path and ergonomics around dynamic segments ending in `model_name_id`? In the case of a `show` route doesn't it automatically call `this.store('model_name', params.model_name_id`? It's a little tip I found delightful as a newbie :smile:


Does it do everything specified?


В данной строке ошибка
Судя по всему должно быть:
$config->db_users_table, $config->db_users_table, $this->prefix
В противном случае ошибки.


see the discussion in #58 , apparently the default in Intellij IDEA is false. What's the problem with changing this default?


what exactly?


guys, can you compiling exe for me to test.


Да, вы правы, спасибо, исправлено в следующем коммите.


line 6512  m_clbSentences->InsertItems( list, m_sentences.GetCount() );

causes an Assert error when list is empty.


Like before, http://jenkins.x64dbg.com see workspace and then look in the
'release' directory

On Sun, 30 Aug 2015 19:41 cxj98 <notifications@github.com> wrote:

> guys, can you compiling exe for me to test.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/x64dbg/x64dbg/commit/f7db29e91ac003cf3bab6603c055722c079a67d1#commitcomment-12966357>
> .
>


A new post with title`"This is a unique post title"` would be created each time `rake db:seed` is run.

To avoid this, so this post is only created once regardless of how many times `rake db:seed` is run, you can use `Post.find_or_create!(...)` instead of `Post.create!(...)`. See the API docs here: http://api.rubyonrails.org/classes/ActiveRecord/Relation.html#method-i-find_or_create_by

(Same applies to `Comment.create!(...)` on  line 11)


bdb is not an actual database :)


Added hint for GLUON_TARGET


See repo https://github.com/StefanScherer/win-patch-iojs for the source code of `patchiojs.exe`.


`Post.find_or_create_by!(...)` and `Post.create!(...)` both return the created `Post` object. Its possible to make use of that to avoid the need for line 9.


Should we always open up githup, or just when we create a new ssh key? For example:

~~~ bash
echo "Checking for SSH key, generating one if it doesn't exist ..."
if [ ! -f "$HOME/.ssh/id_rsa.pub" ]; then
  ssh-keygen -t rsa
  echo "Copying public key to clipboard. Paste it into your Github account ..."
  cat ~/.ssh/id_rsa.pub | pbcopy
  open https://github.com/account/ssh
else
  echo "SSH key already exists ..."
fi
~~~


What do you mean with *"expects the OWIN app to set the headers, which should never happen"*?


測試修訂


@ireneduke these are the changes we talked about last Wednesday. If you see anything wrong with this let me know and I can tweak it some more.


This won't compose with earlier applicative functors' HttpContext values...


This could be reversed to return early, thus avoiding extra indentation of the following code.
Don't do it now. I am going to see the other commits too. If they don't have a problem I'll merge as-is.


This could be done in nix instead of bash since we know the value of the variable as part of the nix expression. This in theory could also prevent a rebuild for linux users.


very minor nit: docstring?


LGTM


# number of particles in each of the directions
variable          Lx   equal 2.0
variable          nx   equal 40
variable          ny   equal ${nx}
if ${nd}==3 then &
"variable nz equal ${nx}"

# size of the domain
variable          Ly   equal  ${Lx}
#if ${ndim}==3 then &
#"variable         Lz   equal ${Lx}"

variable          dx   equal  1.0/${nx}

# density of gas phase
variable          sph_rho_a equal 1.0
# density of droplet
variable          sph_rho_wt equal 0.01
variable          sph_rho_wl equal ${sph_rho_a}

# sound speed
variable          sph_c_wt equal 200/sqrt(${sph_rho_wt})
variable          sph_c_a equal 200/sqrt(${sph_rho_a})
variable          sph_c_wl equal ${sph_c_a}

# viscosity
variable          sph_eta_wt equal 0.069395
variable          sph_eta_a equal 1.0
variable          sph_eta_wl equal ${sph_eta_a}

variable          gamma_wt equal 1.0
variable          gamma_a equal 1.0
variable          gamma_wl equal ${gamma_a}

variable          rbackground equal 0.0

variable          h equal 3.0*${dx}

# mass of droplet particle
variable          sph_mass_wt equal 1.0

# mass of gas particle
variable          sph_mass_a equal 1.5
variable          sph_mass_wl equal ${sph_mass_a}

# surface tension
variable           alpha    equal 150
variable           alpha_awt equal ${alpha}
variable           alpha_awl equal ${alpha}
variable           alpha_wtwl equal 1.5*${alpha}

variable          cv_wt       equal 1.0
variable          cv_a       equal 0.04
variable          cv_wl      equal ${cv_a}

# add an uplift force to the gas particles
#variable          gy    equal 3.60e+05

# type of the gas particle
variable          air_type equal 1
# type of the droplet particle
variable          water_type equal 2
variable          wall_type equal 3


Fixed, thanks chucksk.

Mike


this is definitely always json? or at least, if we get a 2xx?


So we decided that we are not adding translations now. The gui would have a translation framework setup later.


Thanks Jeff, that works.


You should use `change_column`. By removing and adding column you reset all characters exp. Now imagine that app is live and you just removed all exp progress for all users ;)


You do know that xrange is deprecated now in Python 3 as it is the default range function.

You should in your code at the top attempt to map the xrange function to the range name. If that fails then you are running on Python3 interpreter and the range function already is the xrange equivalent. That way your Python2 interpreters will be running the xrange function and your code will just use range which will run correctly on the Python3 interpreters.

Both will run the fastest most efficient range function then... Win Win!


there are creds in here, right? i wonder how easy it would be to sanitize them a la https://github.com/snarfed/oauth-dropins/blob/master/oauth_dropins/models.py#L88 . not sure how consistent we are with that currently in oauth-dropins. not high priority.


oauth1 lgtm :dog: :microscope:


Typo has_may -> has_many


no need for the italics


这里的 `vm.computed` 应该是 `vm.example` 吧？


nm i see it's already done


This is how its done with raw_input and input between Python2 and Python3 interpreters:

# python 2.* compatability
try: input = raw_input
except NameError: pass

Now you can use input function in one code file and it will run correctly on both Python2 and Python3


`Click on the SOOMBOT` replace with `In the games screen`


Property 拼错了 - -


Good research into Faker's API :+1:


@martin-g Cherry picking works like a charm ;-) Thanks a lot. I had some minor issues regarding compiling but I just sent a mail to the mailinglist regarding those ones.


Thank you 😊


its versatility


this update broke my workflow


thanks! looks like you'll probably need to rebase since this already landed in df66a828127dcca335c185455cc27b270b970186


Not yet, it still need to work on. Bryce, I cannot download the lab you attached on the email.. Seems the semo email not works well on zip file. Can you update the code on the GitHub or send it to this email address, yjin1s@live.com


oh nm this is a merge. ok! maybe rebase so this commit disappears? not important.


I just remembered what profile was for. It was so that bz could stick ?profile=1 on the URL and get a version of the spec without the scripts, or some such, for profiling parsing the spec.


I know that you transfered the old code but let's get it right, since you edit this. This overload is [obsolete](http://doc.qt.io/qt-4.8/qmessagebox-obsolete.html#question-4).
Use something like this:
```c++
StandardButton ret = QMessageBox::question(this, tr("Deletion confirmation"), tr("Are you sure you want to delete the selected RSS feeds?"), QMessageBox::Yes|QMessageBox::No, QMessageBox::No);
```


And of course here:
```c++
if (ret == QMessageBox::No)
```


maybe rebase so this commit disappears? not important.


I'd like to ban usage of the left and right inner products on this site please! I have never found them any use in practice, as you can always replace them with a combination of the geometric product and projection. GA should simplify maths, and introducing additional notation makes things worse, IMHO.


-.- 以后直接开 PR 好了


Hello Cédric

Dans login.html à la ligne 11, il y a un tag "label" qui se termine par un tag "input".

Est ce que c'est voulu?


这个我先顺手修了，下次直接开


The package works fine on other architectures.  If you or somebody else wants to provide implementations for other architectures, that would be great, but this commit is not the place to discuss that.


"contain" not "contains"


Thanks for the tips. I'll look into python 3 compat at some stage.
It would be nice to be compat with both, and depending on "six" might help.
Though really python2 and python3 should be considered as separate languages.


@dnsfng Héhé, je me suis planté, je voulais la cacher uniquement sur mobile :)


#4942-VRF-with-water-cooled-condenser-does-not-pass-hard-flow-rates-to-plant-loop (rraustad) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1288 of 1288 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg) ![Coverage Badge](http://img.shields.io/badge/coverage%20status-70.02%25-green.svg)


好


Hello. You are change initialise mechanism of MaynLayout. How to go to Preferences screen on this version?


These two lines need to stay the way they were, as they allow us to get the information about who started a topic and when.


That should be setuptools-scm


В таблицу izdan_type добавить поле weight типа float.


turn right should not return anything


This commit broke something for me.

`11:12 $ python run.py
Traceback (most recent call last):
  File "run.py", line 2, in <module>
    from app import app
  File "/Users/gareth/code/hack-oregon/backend/jobs-economy/app/__init__.py", line 12, in <module>
    from .api import models
  File "/Users/gareth/code/hack-oregon/backend/jobs-economy/app/api/models.py", line 21
    class County(db.Model):
        ^
SyntaxError: invalid syntax`


Line 544 needs to stay the same as well - we're getting info on who started the topic, not who made the most recent post in it.


So I created new migrations from scratch for all apps.


Nice. Good to see this behavior specified by the acceptance tests!


logic incorrect, if secret1 is matched, unlock will be true. you need to match 3 secret and the turn directions inorder to unlock.


shouldUnlock


thx a lot


logic incorrect, if secret2 is matched, unlock will be true. you need to match 3 secret and the turn directions inorder to unlock.


<!-- PMABOT:SOB -->
This commit is missing Signed-Off-By line to indicate that you agree with phpMyAdmin Developer's Certificate of Origin. Please check [contributing documentation](https://github.com/phpmyadmin/phpmyadmin/blob/master/CONTRIBUTING.md) for more information.


Why return dial?


Glad to see experimentation with `before_action` via the `set_question` private method.

For some developers they find this usage of populating the most important instance variables in `before_action`s hard to follow - it can be a bit of a (easily solved) mystery where `@question` comes from. Others like it.

Make up your own mind about whether you find this DRYing up keeps the code easy enough to follow.

There's a middle ground which is to tweak this so you call `@question = find_question` as the first line in each of the dependent actions. That way you get the DRY benefits and you make it clear for a developer to follow the steps executed in each action.


who needs to set secret?


who needs to get secret?


Good to see `set_question` not exposed as a public method, :+1:


Not really. These changes are mostly compiler fixes and porting of graphics code but all module loading for Lua is still not ready.
I tried to run it on the Android emulator and it worked for some small test but it crashes when I load it on my tablet. There needs to be some debugging to tell why, didn't get around to do that though.

https://github.com/NeoGameEngineProject/NeoOnAndroid ==> Here is the install and compile script for building the *.apk


If I am not wrong `ipairs` are known to be "slow". Can we keep a `for` or `while` style here?


No worries, I just hate to see new effort done at this day and age that makes hardcoding Python2 changes that then make it more work effort to convert to Python3 tomorrow so I had to pass on the tips.


Hey Johnny!

Awesome, I was looking for a python web app starter. Learning this would be fun. Thanks for starting it.
Keep up the great work.

Ravi


@bzbarsky do we need to reinstate that somehow?


how about: this.dial = (this.dial + ticks) mod 40


I didn't know string supported templating-like functions. Is the checksum whitespace sensitive ?


Please address every comments I have.


If you've not encountered these yet, consider taking a look at Devise's `:lockable` and `:timeoutable` modules so you're aware of what they do. You'll be able to make use of them in the projects phase if you're interested.

You can find out more about them from the summaries in the bullets near the top of [Devise's README](https://github.com/plataformatec/devise), and following the links next to them to see the API docs which will give you a bit more detail.


Yes, but it shouldn't modify the string at all, look at how the optionals
are already working in that expression.

On Sun, Aug 30, 2015, 11:21 zimbatm <notifications@github.com> wrote:

> I didn't know string supported templating-like functions. Is the checksum
> whitespace sensitive ?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/zimbatm/nixpkgs/commit/782faa742498aeae8eb2512b81878f25693408b4#commitcomment-12966524>
> .
>


If the first test passes then this test must also pass.


Please reset the changes to formatting. The original formatting was right.
Right:
```
public void method() {
    if () {
        ...
    } else {
        ...
    }
}
```


Likewise, if the first test passes then flipping a bit in the result trivially gives a different value.


I'm here for the same reason


logic incorrect, in your implementation, I can never turn right again.


A slightly more descriptive reason for pull request


<!-- PMABOT:SOB -->
This commit is missing Signed-Off-By line to indicate that you agree with phpMyAdmin Developer's Certificate of Origin. Please check [contributing documentation](https://github.com/phpmyadmin/phpmyadmin/blob/master/CONTRIBUTING.md) for more information.


@dnsfng Yes j'avais vu, j'ai passé un fix


While I'm on the subject of Python2 and Python3, can you fix the shebang in the fslint-gui.

You can always specify the interpreter to run using python2, python3, etc. by you are shebanging the env python which has switched from python2 to python3 in modern distro's and you are breaking on installation of fslint on those modern distros.... just shebang for the system's python2 interpreter instead of the generic python interpreter and you'll fix all that breakage on those systems. (Issue #111)


Hey Abhi, it's on [line 463](https://github.com/jraman/gpupetuum/pull/8/files#diff-005bc65092817d05e5c82bfa6110dd42R463).  Search for "sqrt".


Not sure why the diff is weird, but all I did here was shift everything 2 spaces to the left.


Hahaha, thanks man! Means a lot.
On Aug 30, 2015 12:33 PM, "arciel" <notifications@github.com> wrote:

> This recursion graphic is better than the ones I've seen in textbooks and
> tutorials.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/TusharRakheja/TicTacTongue/commit/b7b3c294a9119adb706d77f8f18e1956dff0a255#commitcomment-12966315>
> .
>


Kesley concerte aqui


I didn't investigate further yet, but:

 * Checking app-office/skrooge-2.0.0 for undeclared dependencies
 *   /usr/lib64/qt5/plugins/skg_debug.so links to /usr/lib64/libKF5TextWidgets.so.5
 *     Missing DEPEND on kde-frameworks/ktextwidgets
 *     Missing RDEPEND on kde-frameworks/ktextwidgets
 *   /usr/lib64/libskgbankmodeler.so.2.0.0 links to /usr/lib64/libKF5Service.so.5
 *     Missing DEPEND on kde-frameworks/kservice
 *     Missing RDEPEND on kde-frameworks/kservice
 *   Suspect RDEPEND: dev-libs/libxslt dev-libs/qjson dev-qt/designer dev-qt/qtnetwork kde-frameworks/kdesignerplugin kde-frameworks/kf-env kde-frameworks/kguiaddons kde-frameworks/kjobwidgets kde-frameworks/kwindowsystem x11-misc/shared-mime-info


I love the way the fall death looks Jason. The camera stopping is a nice effect.


Apparently commit hashes get removed from commit messages. Here's the full message.
```
Removed due to the fix from commit `bf9c7697`
```


Do not create a single instance for all your tests. You should create a instance for each test to eliminate interference.


Ui vorsicht: hesch bewusst "{ }" weg gla? Mit nume einere Zeile funktioniert das scho, aber ich empfiehle, immer Chlammere z'mache :-)


Hi,
@00svbeon
I've been absent for a while. I've just seen that you delete and after reset the french translation.
That's unfortunate. :(
Is there a specific reason or can I re use this version?


Володимир, зрозуміло, дякую за пояснення.

Я так розцмію що ця зміна не є особливістю українського перекладу. Чи є?

Якщо це дійсно корисна зміна і вона стосується всіх мов то пропоную
відкрити PR на https://github.com/progit/progit2 натомість і там можна
більш компетентно обговорити цю пропоновану зміну.

//taras

Я додав команди для зручності компіляції книги локально у потрібний формат.
Вливати ці зміни у головну вітку чи ні - це на Ваш розсуд.
Стандартна команда build генерує книгу в усіх форматах. Мої команди
дозволяють згенерувати тільки у потрібному форматі, на стандартну команду
це не впливає. Я фактично розбив підкоманди згруповані командою build на
окремі групи, що стосуються конкретного формату.
Я раніше також ніколи не писав на Ruby, але мої зміни не містять складної
логіки і є просто логічним наслідком з уже існуючого коду.

—
Reply to this email directly or view it on GitHub
<https://github.com/VBoden/progit2/commit/105b8f1985cb51d360cf32a30b32e0bbc054caff#commitcomment-12966147>
.


we need a spelling bee.

assests ... assets


I tested it successfully here and it works great with xl-320! I also added support for high dpi monitor today, the gui is displays better on retina monitors now.


turn() should return nothing.
You should have a method in Bug called getDirection() that returns ENUM for LEFT or RIGHT


plot speedup function added with a picture of the plot


Hi. Thanks to your contribution. It however does not work when the last slide is marked as COMMENT.


@garfield69 Hi, I rewrited the french translation but it took me about a month so there are now some line differences between fr_fr.json files.
How do I updated my file to match the current one while keeping my translation improvements?
I'm new at this...
Thanks


ok


this regex is called is2014, and it checks for 2015. It is used later on as is2015 on line 22, which is undefined as far as I can see


agree


apoap?


using System;
using System.Collections.Generic;
using System.Linq;
using Microsoft.Xna.Framework;
using Microsoft.Xna.Framework.Audio;
using Microsoft.Xna.Framework.Content;
using Microsoft.Xna.Framework.GamerServices;
using Microsoft.Xna.Framework.Graphics;
using Microsoft.Xna.Framework.Input;
using Microsoft.Xna.Framework.Media;

namespace WindowsGame1
{
    public class Game1 : Microsoft.Xna.Framework.Game
    {
        GraphicsDeviceManager graphics;
        SpriteBatch spriteBatch;
        Texture2D image;
        Vector2 pos;
        Vector2 speed = new Vector2(100, 100);

        public Game1()
        {
            graphics = new GraphicsDeviceManager(this);
            Content.RootDirectory = "Content";
        }

        protected override void Initialize()
        {
            // TODO: Add your initialization logic here

            this.IsMouseVisible = true;

            base.Initialize();
        }

        protected override void LoadContent()
        {
            // Create a new SpriteBatch, which can be used to draw textures.
            spriteBatch = new SpriteBatch(GraphicsDevice);

            image = Content.Load<Texture2D>(@"image");
            // TODO: use this.Content to load your game content here
        }

        protected override void UnloadContent()
        {
            // TODO: Unload any non ContentManager content here
        }

        protected override void Update(GameTime gameTime)
        {
            // Allows the game to exit
            if (GamePad.GetState(PlayerIndex.One).Buttons.Back == ButtonState.Pressed)
                this.Exit();

            // TODO: Add your update logic here

            MouseState mouse = Mouse.GetState();

            if (mouse.LeftButton == ButtonState.Pressed)
            {
                int x = mouse.X;
                int y = mouse.Y;
                pos = new Vector2(x, y);
            }

            pos += speed * (float)gameTime.ElapsedGameTime.TotalSeconds;

            int max_x = GraphicsDevice.Viewport.Width - 35;
            int max_y = GraphicsDevice.Viewport.Height - 36;

            if (pos.X > max_x || pos.X < 0)
            {
                speed.X *= -1;
            }

            if (pos.Y > max_y || pos.Y < 0)
            {
                speed.Y *= -1;
            }

            base.Update(gameTime);
        }

        /// <summary>
        /// This is called when the game should draw itself.
        /// </summary>
        /// <param name="gameTime">Provides a snapshot of timing values.</param>
        protected override void Draw(GameTime gameTime)
        {
            GraphicsDevice.Clear(Color.CornflowerBlue);

            // TODO: Add your drawing code here
            spriteBatch.Begin();
            spriteBatch.Draw(image, pos, Color.CornflowerBlue);
            spriteBatch.End();

            base.Draw(gameTime);
        }
   }
}

Just use your own image and adjust the height and width according to the image. My image was 35 x 36.


Just an FYI - After http://patchwork.ozlabs.org/patch/512137/ is merged, LDFLAGS constructs like this will no longer be necessary. I am putting together a list of packages affected by this change to nls.mk and will be submitting updates after my testing is complete. You have a few (like this one) that will be affected.


Linux 4.2


Everything on it works. The only thing is that it creates an image to start before I click the mouse button. Not sure if that's a bad thing or not. It still makes an image where I click the mouse button.


missing "declares"


Why is bdb there then?


I forgot to rename from Mod to ChunkHanlder


This is the part that I think is no longer required.
As far as I know, markdown supports both code block formats, at least the online editors that I tried show no difference. But using both formats can lead to errors in formatting.


muchas gracias por responder, pero pudieras decirme donde puedo
documentarme para hacer eso?

2015-08-29 23:52 GMT-05:00 sabik <notifications@github.com>:

> No es un plugin es una API que corres en tu servidor y luego la usas con
> llamdas GET de HTTP, usando url_open, file_get_contents, curl o similar
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/mexicapis/estafeta-api/commit/97fca0c4d42cf116e3137dd2c3223a7b47dfd9f2#commitcomment-12963983>
> .
>


This was forked from Sling?


Thanks for the catch!


This commit is the part I find most important.
Please remove the other commit and I'll merge.


Thanks. I enjoyed implementing it :)

Can someone invite me to Trello. I don't know what to work on, so just
spent a lot of time looking into UI stuff...
On Aug 30, 2015 2:34 PM, "Jacob Millner" <notifications@github.com> wrote:

> I love the way the fall death looks Jason. The camera stopping is a nice
> effect.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/SideTrackedStudios/NutJob/commit/60c21580aefffbf0f7c7d4ef45357d9b6d208c94#commitcomment-12966566>
> .
>


I like this


@Nimrodsv


const std::string&


Revise according to the above comment (somehow incorporate declare which player will be attacked).


I think we want to do `return self.__initGame()`, so that the user is just re-prompted instead of exiting, right?
Good catch on `__startGame()`, though. That definitely wasn't a valid function.


@tohojo: this is just a proposal, just nuke it if you do not like the almost non-existent behavioral improvement...


c'est normal que tu aies 2 fois var creditbutton, et creditbuttontext?


This commit breaks compilation under Linux since it attempts to use an undefined variable "nullptr" as seen below:

[ 47%] Building CXX object src/bnetd/CMakeFiles/bnetd.dir/file.o
/data/root/pvpgn/pvpgn/src/bnetd/file.cpp: In function ‘const char* pvpgn::bnetd::file_get_info(pvpgn::bnetd::t_connection*, const char*, unsigned int*, pvpgn::bn_basic (*)[8])’:
/data/root/pvpgn/pvpgn/src/bnetd/file.cpp:111:27: error: ‘nullptr’ was not declared in this scope
    const char *filename = nullptr;
                           ^
src/bnetd/CMakeFiles/bnetd.dir/build.make:514: recipe for target 'src/bnetd/CMakeFiles/bnetd.dir/file.o' failed
make[2]: *** [src/bnetd/CMakeFiles/bnetd.dir/file.o] Error 1
CMakeFiles/Makefile2:979: recipe for target 'src/bnetd/CMakeFiles/bnetd.dir/all' failed
make[1]: *** [src/bnetd/CMakeFiles/bnetd.dir/all] Error 2
Makefile:127: recipe for target 'all' failed
make: *** [all] Error 2


Does this even compile?


Ah, I see it does have a lot of built-in leading. I was using two separate text lines. It looks like it's 50/53pt (with 30/53pt for the punctuation), if that helps. Let me know if there's anything else I can provide that might help!


I'm aware of this and it will be fixed in the next commit. The fix is to add the C++11 or C++14 flag to your compiler: http://forums.harpywar.com/viewtopic.php?pid=7255#p7255


Should be on 2 lines, not 4.


Add blank line between this and BUILD_REQUIRES.


Unfortunately, i can not as it may contain confidential information.

Using artisan, you can seed a default database, here is more information about it :
http://laravel.com/docs/5.1/seeding#running-seeders

I would be glade to get a pull request, providing a proper seeded database to the community.
For additional information, check the following link: https://github.com/julien-amar/crm-laravel/tree/master/src/app/database/seeds


This goes between PROVIDES/REQUIRES and BUILD_REQUIRES/BUILD_PREREQUIRES.


IMHO, information about last poster more interesting in last topics list.


Yeah, that's fair. I only changed it to sys.exit(1) because I wasn't sure what to do.


OK, thanks for letting me know.


Not an issue but IMHO the old structure was more readable.


One occurrence of `img` should be replaced to `img_src`.


Так, ця зміна не є особливістю українського перекладу. Можливо це й не дуже корисна зміна.
Подивився на саму книгу на сайті - побудова виконується добре і без моїх змін. Моживо ці зміни взагалі не вливати у головну вітку, а десь відзначити, що потрібно зробити, якщо будуть проблеми при побудові pdf-файлу на локальному комп’ютері. Раптом хтось іще захоче локально збудувати книгу))) (наприклад, відкрити issue в офіційному репозиторію перекладу)
І прошу вибачення, що все ж таки змусив потратити час на перегляд зроблених змін


Hmm, ok that would "work",  but why change the coding standards when its better to just fix the cause of the problem?  Not sure I get why the coding standards would need to be changed.  It seems to me like that would allow for sloppy coding.


This also fixed the "value=Resource not found for the segment 'undefined'." issue i had when using manager.createEntity with WCF.

POST Before fix

    --batch_5e1d-6113-020e
    Content-Type: multipart/mixed; boundary=changeset_d0b9-460e-2770

    --changeset_d0b9-460e-2770
    Content-Type: application/http
    Content-Transfer-Encoding: binary

    POST undefined HTTP/1.1
    .......

POST After Fix

    --batch_1f93-f216-1223
    Content-Type: multipart/mixed; boundary=changeset_9fa4-7575-a231

    --changeset_9fa4-7575-a231
    Content-Type: application/http
    Content-Transfer-Encoding: binary

    POST Authors HTTP/1.1
    .......


This should be `*int` because it is optional.


`$lang` was defined [here](https://github.com/phpbb/phpbb/commit/9486c693b649f4e7e412dd5c8cf9c247507a7f29#diff-9df17e3632600c7fffc68f5d137b8734R534).
If MCP variables are already included in the user object, then [this](https://github.com/phpbb/phpbb/commit/9486c693b649f4e7e412dd5c8cf9c247507a7f29#diff-9df17e3632600c7fffc68f5d137b8734R533) and [this](https://github.com/phpbb/phpbb/commit/9486c693b649f4e7e412dd5c8cf9c247507a7f29#diff-9df17e3632600c7fffc68f5d137b8734R534) lines should be removed.

But I wonder if there's a reason why they were included here before.


Hi cool,
but with this change, you drop support for Qt 4.8 and mandate Qt5. maybe you should add the code under conditionnal directive if cmake detects Qt 4.8 or 5.

thanks.


`deployment version must be greater than 0`


Oui mais je dois d'abord corriger mon dépôt selon tes conseils.
J'ai pas eu la force cette semaine d'y travailler, je fais ça bientôt.


Version should be optional.


You should be using the object serialization method that cesar used for other deployment log types - run the conversion from api.DeploymentLogOptions to `map[string][]string`


This should be optional


Why aren't you doing this server side?


By the way, Doug, wouldn't you also want to maintain sleepd, which shares
lots of code with wmbattery?

On Sun, 30 Aug 2015 13:53 Doug Torrance <notifications@github.com> wrote:

> Oops, I completely missed that. Thanks for pointing this out!
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/d-torrance/wmbattery/commit/9f7be30fc6a2541a89e129b82d41c6f8d2c0d71a#commitcomment-12965208>
> .
>


Should be server side, and be a 204 no content


The punctuation and the DE should be the same styling - color and size - at least the way I have it in the design. Don't know if that can help simplify anything. Also just to be annoying, the spaces around the : and DE are the smaller font size so that the numbers aren't so far apart - but the space for the date and after the comma are the full size (or two of the small size).


Will do!


```.where``` will return a collection. We probably want to return a boolean value here. ```record.user == user``` should work here.


s/git/tig/


the last element may be a comment, so let's index from the beginning instead: [DOT, method name, expression list]


Why is this being done twice - once here, and once in REST.Get()?  When will this happen?


This can panic and therefore requires a `util.HandleCrash()`


whoops, fixed.


This should be returning a 204 no content.


Should be a 204 no content.


Don't embed clients like this - they are not part of the public interface.


Added `changeExt` option to avoid changing file extension to .html (Advanced use cases) , `changeExt` defaults to true so normal cases will change ext to .html and for those who need can avoid changing extension


You should look at it as: the cause of the problem is that the flag isn't set.

The new C++11 standard has very useful features and should be utilized now that it's been 4 years and compilers have support for it. Staying on C++99 unnecessarily restricts the project to the old ways of coding.

Specifically in this case, ````NULL```` is the usually equivalent to an int of value 0, which doesn't make sense when a type of char is the type and it is a pointer.


This doesn't need to be a member of REST - it should be a private function that takes a timeout and a client.


Ah, det visste jeg ikke. Da legger jeg den tilbake :)


Ok, but... It's very strange. Method/property visibility helps you to control that no one may change behaviour or rely on this behaviour of some particular method. In this case when you need refactor(or even delete) such method you do not need to worry about any BC breaks. This library is popular so BC is important.
Anyway you use some notation that helps divide methods/properties by visibility. Why not use native PHP construction?
Also I have reviewed ticket https://github.com/phpseclib/phpseclib/issues/575 and found another strange thing. Your notation is useless until you're suggesting to other people use protected methods:
http://stackoverflow.com/questions/4484246/encrypt-and-decrypt-text-with-rsa-in-php/4534385#4534385
So `_exponentiate` became `public`, but now you can't rename it to `_exponentiate` because it will introduce BC break due to someone already use it with old name `_exponentiate`. The only way is to create another method `exponentiate` that will call `_exponentiate` until BC will be dropped in some version.


He's not breaking standard, he's using the new standard.  C++11 is quickly becoming the "right" way to do things.


Should we add ```:update``` to except as well?


It should also be `version`


I understand that he's not "breaking" the c11 standard.  But what I am questioning is why is the standard being changed?  If the rest of the project conforms to C++99,  why are we changing it?  I dont remember seeing any discussion about changing to c11.  I think thats what I'm mystified about.


ACK - tested locally and it's clearly the right thing to do.


Plugin.yml resides in src/resources. If you build using Maven, the plugin.yml should appear in the jar. Please tell me if it doesn't.


локатор можно сократить до "//tr[@name='entry']"


Зачем зкомментировали?


Are you sure?

Please note the run index i changed direction, which causes different results because of  rightshifting tva in the loop.

In my tests on an amd64 [little-endian architecture], the patched version converts 0x3837363534333231 to
- "87654321"    in the !WORDS_BIGENDIAN case [unchanged]
- "12345678"    in the WORDS_BIGENDIAN case
[using the fact that  0x3N = "N"  for 0 <= N <=9]

To me that looks correct, because for a big-endian architecture the strings would end up reversed, making the the big-endian case end up exactly like the little-endian case on the little endian architecture (i.e. "87654321")

What issues do you see exactly?


Make this public


I would just make install require web service.  It doesn't make sense to have this method return it.


Hmmm? I wrote the DemoAutonomous program and added it; did I just phrase it wrong?

-Matthew Thompson

> On Aug 30, 2015, at 10:55 AM, Jacob Mason <notifications@github.com> wrote:
>
> This might be a mistaken addition.
>
> ―
> Reply to this email directly or view it on GitHub.
>


Is this the behavior we want:

    if no subdomain
       redirect to 2015
    else
      next


I would make the existing WS an optional argument to InstallREST.


>But what I am questioning is why is the standard being changed?

There are new, useful features that should/could be utilized.

>If the rest of the project conforms to C++99, why are we changing it?

This project is huge, the rest of the project shouldn't be a reason why we shouldn't change the standard. Even the transition from C to C++ has **barely** progressed even though it has been years now. I didn't change other functions because I was already modifying the function that I added ````nullptr```` to, so killing two birds with one stone here.

>I dont remember seeing any discussion about changing to c11.

Correct, there was no discussion, but it was because I didn't think anyone would object to using the C++11 standard. It was an error on my part to not add the flag to begin with, I test with Visual Studio on Windows and was not aware that the flag must be explicitly set.


^ @indexzero


Exactly my point.  You cannot simply switch "standards" just because it suits your coding style.  When you attempt to merge 2 or more different coding "standards" into 1 project without telling the other people about it, HUGE problems can arise that are not spotted for weeks, months, even years perhaps.  And thats a pretty bug "opps".


LGTM.  I'm not super happy about the _CHECKSUM suffix when the other quantities have a _CRC suffix, but that's just a personal preference.

Also, I can't help but feel that the proper solution is to make all ERTS properties into runtime parameters, as this whole "encode some but not all ERTS properties into the HiPE compiler's .beam files" has bitten us more than once.  But that's clearly for a future cleanup.


:D


Added this as a note for whether to address non-200 status codes as errors, rather than allowing 100, 200, and 300 status codes.


I'm not aware of any errors that would silently occur when transitioning from C++99 to C++11. All existing code is C++11 compliant with very few exceptions such as the now-deprecated ````autoptr````.


Yours is much better than what I got right know. I think start before mouse click will be ok, and we still have time think about it. Let's wait Yongqi and Wenxing's response.


No, this is just a one-line file and I don't know why it's here.


@wkennington like this ? c4b4121d909142c9a84d43f8d0fbf94aa2f7c536


It can't go in couldMatch because it doesn't have an object at that point, so it has to go in matches.


:exclamation: Did this just drop the second value? Seems like you either need a do/while loop, or add the following before this loop.

```csharp
result.Append(separator);
result.Append(en.Current);
```


Exactly,  errors that havent happened yet are quite silent.  Before you go changing the standard,  a good amount of discussion should happen before that happens no matter how "seamless" the transition "should" be because lets face it,  this is development.  Nothing goes seamlessly ;)  I propose that you fix the issue properly (it's not that difficult anyway so whats the issue).  And then bring up a discussion about changing to C11 instead and have it be its OWN commit rather than hiding it as a fix to some unrelated problem.


Can this be merged ?


Bad regex.


TeamCity pull requests :: netty [Build 151](https://garage.netty.io/teamcity/viewLog.html?buildId=992&buildTypeId=pulls_netty&guest=1) is now running


Wat.


Why not just use the entity tag?


test


Това можеш да го изключиш с `.rubocop.yml` файл :smile:


Do we need `super` here?


that was fast, thanks a lot!


@ProjectcoreDevs, they still looks wrong http://prntscr.com/8apka0


I don't believe we do...but it also doesn't actually hurt anything.


first change


Yeah

On Sun, Aug 30, 2015, 12:39 zimbatm <notifications@github.com> wrote:

> @wkennington <https://github.com/wkennington> like this ? c4b4121
> <https://github.com/zimbatm/nixpkgs/commit/c4b4121d909142c9a84d43f8d0fbf94aa2f7c536>
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/zimbatm/nixpkgs/commit/782faa742498aeae8eb2512b81878f25693408b4#commitcomment-12966788>
> .
>


I meant that this lens, as previously written, appears to expect the OWIN app to replace the headers dictionary rather than merely update the values in the headers dictionary. I think you need a custom dictionary here that responds to `IDictionary.Add` by calling the function in the setter part of the lens. That may be the intent of the `Muuutation` type, but I didn't see that occurring, and the setter path is never called.


I'm not surprised. I was just making enough changes to get the OWIN sample to work.


Hey stop making garbage this is just a bunch of crap isn't it? you just look like a nooooooob bro


This is not ideal, as this approach prevents any form of streaming. I think a custom stream type, as noted above, is a better solution.


I copypastaed that from another event....


This should probably be our remote (as this is our contributing guide)


A link to the documentation would be nice here :smile:


Cause... I didn't know we had that tag... >.>


This change broke backwards command line compatibility.  You can't do that without providing a deprecation notice and backwards compat.  This needs to be fixed before we cut 1.0.6, which will be soon.  Please provide a PR to restore --portal-net than marks it deprecated.


Or at least git told me it was different. Strange


A pull request is very rarely initially accepted? (however almost all pull requests are accepted after some work)


컴파일러가다른건가... 왜 띄어쓰기가 이렇게되지


TeamCity LabkeyTrunk :: Git postgres 9.4 [Build 461](https://teamcity.labkey.org/viewLog.html?buildId=269165&buildTypeId=LabkeyTrunk_GitPostgres) outcome was **FAILURE**
Summary: Tests failed: 5, passed: 0 Build time: 00:08:50
### Failed tests
```
org.labkey.test.Runner: org.labkey.test.tests.CDSTest: <no details avaliable>

org.labkey.test.Runner: org.labkey.test.tests.CDSLoginTest: <no details avaliable>

org.labkey.test.Runner: org.labkey.test.tests.CDSMeasureStoreTest: <no details avaliable>

org.labkey.test.Runner: org.labkey.test.tests.CDSVisualizationTest: <no details avaliable>

org.labkey.test.Runner: org.labkey.test.tests.CDSSecurityTest: <no details avaliable>

```


Calling back() after pop_heap seems a little awkward.
It also loses a chance to combine code for the normal case and the "one partition left" case above.
Instead, do the following before the "if":

            range_iterator rangeIt = partitions.begin();
            tuple = *rangeIt->first;
            assert(rangeIt->first != rangeIt->second);
            ++rangeIt->first;

Then,

            if (partitions.size() == 1 && rangeIt->first == rangeIt->second) {
                partitions.pop_back(); // done.
            }
            else if (partitions.size() > 1) {
                std::pop_heap(partitions.begin(), partitions.end(), tupleRangeComp);
                partitions.pop_back();
                if (rangeIt->first != rangeIt->second) {
                    partitions.push_back(*rangeIt);
                    std::push_heap(partitions.begin(), partitions.end(), tupleRangeComp);
                }
            }


Right, I think you're right – you likely need object.m from a more recent libdispatch that declares an Objective-C class, so that calls to objc_{retain,release} emitted by the compiler work on dispatch objects.


@Pathoschild : Unfortunately this name can't be used in user scripts, due to [T96901](https://phabricator.wikimedia.org/T96901). Here is a minimal example which causes
> JavaScript parse error: Parse error: Unexpected token; token 3 expected in file 'User:He7d3r/global.js' on line 6

https://meta.wikimedia.org/w/index.php?title=User:He7d3r/global.js&oldid=13382609


Let me know how you go, I'm interested in porting NeXTBSD's libxpc to Windows and that may benefit from a more recent libdispatch.


`static this` code will run every time a new thread is created, perhaps this should be `shared static this`?


Add comment: // Should only be NULL when unit testing.


Yes. Thanks.


thanks


:D
![a275a5c7580](https://cloud.githubusercontent.com/assets/3467784/9565391/f7a743e8-4efe-11e5-9111-f1e2c52a36ce.jpg)


add one more word


Add few words


Hello levibostian.

Thank you very much for sharing your modified Mitro-Android source code.

It seems that the port number of remote host is set to be 5005  instead of 443 suggested by Mitro team.

mitro-core/android/MitroApp/.idea/misc.xml

+      <option name="HOST" value="localhost" />
+      <option name="PORT" value="5005" />

Did you log in to the self-hosting Mitro server successfully? Thank you.

Best regards,

Yang Hong


in general, it's best to create an independent test.  It makes the multi-version test a bit odd in windows.


This output line is redundant.
The debugInfo caller in AbstractPlanNode has already identified the plan node type.


Ga perlu function, cukup object biasa sudah ok


Eh tapi ini kan dia dipanggil initModule(forge)

Jadi sebaiknya cari tempat si modul ini dipanggil, dan suntikkan PKIWebSDK.private.forge alih-alih forge aslinya.


sorry not sure if these are actually free


这里直接判断testUrl,应该会导致外链全部被替换吧..万一外链是个rmtp就会出现问题.当然加判断是好的,不过我觉得不应该加在最外层吧.
后面的几个同理


Line:417 以及后面的几处
直接判断testUrl,应该会导致外链全部被替换吧..万一外链是个rmtp就会出现问题.当然加判断是好的,不过我觉得不应该加在最外层吧.


새 브렌치에서 작업 하랬더니?!


@koistya What you have should work, however, this will create a new connection pool every time your code is run, which has its' own resource overhead... it's usually best to create a `connection` instance (pool) once per database in your application, and only call `close()` when you are wanting to exit your application.


This commit might cause a current issue about compass which a pull request is able to fix: 6d6ecb84736766d46a03697165b6320a263d006d


Cool, it works


ok, good


NoClassDefFoundError, AppCompatDelegate.Why?


어처피 이건 지금프로그렘에 적용해야하는거라서 적용했음.
새브렌치에서 작업할건 이제부터 해야될 바이너리 패치


프로그램을 이해하고 짜도록... 잠시 권한 박탈.....


아닛!?!? 문제없는데!?!?


기능은똑같은데 최적화작업을 진행했던것 뿐임 ㅠㅠㅠㅠㅠ


no lo probe, pero esta accion ya la contemple dentro de listenerOnAttack


Noice ;-)


Couldn't get the destroy test to pass (the create one above was easy). I keep getting the error that it can't find the id of the anchor tag that I'm trying to click using Capybara. Any idea why? I tried both using Xpath query, as well as the click link with Ruby interpolation (`item-#{item.id}`). The ID is created in the `items/item` partial.


그리고 Sleep(100)으로인한 리소스차지. 그리고 자막의 시간차를 수정하기위해서 뮤텍스로 펄서를 만든거고


@waddlesplash good catch, i think this was just a git rebase merge mistake.


existen minimas diferencias entre uno y otro pero podemos analizar de crear un "util" para estas cosas o bien moverlo a abstractEvent


es cierto...solo es una costumbre mia de nombrarlos asi.


no para mi...al crear los teams cada uno ya tiene predefinido un color pero esto me lleva a que solo se puedan crear 9 teams como maximo q en su momento me parecio mas q suficiente....aun asi de esta manera al crear el team no necesitas preocuparte por definirle el color y q este no se repita. entre otras cosas


Looks like you misspelled "canola"!


Acid part:
http://paste2.org/XZUdHxbE


es cierto, pero no quize meterme mucho en este codigo almenos en este pr.


Sure, I'll look into it.


this was only changed in the unminified compiled file- not in the source


 "function defaultValue($default)" - do the same thing as your new function
 "function defaultExpression(Expression $default)"

My point is that new function should be like:
```
   /**
     * Specify the default value for the column as an Expression, like 'NOW()'.
     * @param string $default the default value.
     * @return $this
     */
    public function defaultExpression( String $default )
    {
    	$default = new \Yii\db\Expression ( $default );
    	$this->default = $default;
    	return $this;
    }
```


TeamCity LabkeyTrunk :: Git postgres 9.4 [Build 463](https://teamcity.labkey.org/viewLog.html?buildId=269171&buildTypeId=LabkeyTrunk_GitPostgres) outcome was **FAILURE**
Summary: Tests failed: 5, passed: 0 Build time: 00:07:36
### Failed tests
```
org.labkey.test.Runner: org.labkey.test.tests.CDSTest: <no details avaliable>

org.labkey.test.Runner: org.labkey.test.tests.CDSLoginTest: <no details avaliable>

org.labkey.test.Runner: org.labkey.test.tests.CDSMeasureStoreTest: <no details avaliable>

org.labkey.test.Runner: org.labkey.test.tests.CDSVisualizationTest: <no details avaliable>

org.labkey.test.Runner: org.labkey.test.tests.CDSSecurityTest: <no details avaliable>

```


이해했다.. 다시 권한 할당


이사람아!!! ㅠㅠㅠㅠㅠㅠ 왜그레 ㅜㅠㅠㅠㅠㅠ


TeamCity LabkeyTrunk :: Git postgres 9.4 [Build 462](https://teamcity.labkey.org/viewLog.html?buildId=269169&buildTypeId=LabkeyTrunk_GitPostgres) outcome was **FAILURE**
Summary: Tests failed: 5, passed: 0 Build time: 00:10:18
### Failed tests
```
org.labkey.test.Runner: org.labkey.test.tests.CDSTest: <no details avaliable>

org.labkey.test.Runner: org.labkey.test.tests.CDSLoginTest: <no details avaliable>

org.labkey.test.Runner: org.labkey.test.tests.CDSMeasureStoreTest: <no details avaliable>

org.labkey.test.Runner: org.labkey.test.tests.CDSVisualizationTest: <no details avaliable>

org.labkey.test.Runner: org.labkey.test.tests.CDSSecurityTest: <no details avaliable>

```


This is necessary. Otherwise, the friend declaration declares a nested struct that is a friend, unrelated to the one at namespace scope. The fact that the code doesn't error as it is seems to be a bug with gcc's access checking.


But why that was enabled before?


메뉴얼훅 코드를 업데이트했습니다!


:heart:


You need to remove the backup files i.e. #practica1#1# and practica~


Kevin likes heredoc with `<<-SQL` so his editor does syntax highlighting


Too many options, so they should rather be groupped:
SIGNUP_VERIFY_EMAIL_CHANGE = {'start': (title, body), 'continue': '', 'finish': ''}
or even replace all that stuff by template names used by ``django-sitemessage``


Always try to check the base case


Note that for the user all settings start with SITEGATE_. Inner constants do not use this prefix.


This duplicates ``foo`` example above?


thanks


Since PocketMine uses it, I thought I would use it. Afterwards Shoghicp
told me not to use them, as they add lots of overhead. Anyway, they never
worked for me.

On Saturday, August 29, 2015, The Evil Man <notifications@github.com> wrote:

> But why that was enabled before?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/DragonetMC/Dragonet/commit/8ca99d98b6d97edcde470a67ef487091ddd4f2c2#commitcomment-12963713>
> .
>

--


앗 넵!


use**r**


a dozen undocumentedがかかるのはmini-filesystem-librariesでは


Расскажи, что ты тут сделал?


If you're going with this approach, you'll need to separate subtraction from addition as well.  It currently behaves in an inconsistent way: `1 / nested(2 - 3)` -> `1/(2 - 3)` but `1 - nested(2 - 3)` gives `1 - 2 - 3`, which is inconsistent with the division case.


Cool, I'll take a look at it.  Thanks for the heads up.


>Before you go changing the standard, a good amount of discussion should happen before that happens no matter how "seamless" the transition "should" be because lets face it, this is development.

I believe I'm the only developer left on this project, so there will be little or likely no discussion at all if I open an issue asking for comments on switching to C++11.

> I propose that you fix the issue properly (it's not that difficult anyway so whats the issue).

Another commit will be here soon to add the C++11 flag.

> have it be its OWN commit rather than hiding it as a fix to some unrelated problem.

As I said, it was a two birds and one stone situation. Creating commits for the sake of putting a few ````nullptr````'s here and there and titling it "C++11 [...]" seems to be a bit excessive for little benefits. Maybe it would be better to just reference the commits in a new issue instead.


@rwinch I believe that this class would be better to part of Spring Framework. What do you think?

I had opened an issue related to this:
https://jira.spring.io/browse/SPR-11659


Awesome!!!


first 3 maps*
shhh


Is your theme excluding HC in some other way?


All this shell code for supporting ``r940`` should be dropped. Including the patch files.


No, we don't use it anymore. https://github.com/wpsitecare/carelib is somewhat of a HC fork and doesn't need to live in the site root. It also doesn't do anything special for textdomains so it doesn't need to be excluded.


Looks like a fluke, clearly forces in `curl` and the browser.


thx


Are you still working on this?


I really like what you did with variable names! `upr` and `lwr` are very descriptive. :yum:

But what I am looking for here is to store a character literal in these values like:
```
char uprH = 'H';
char lwrU = 'u';
```

Do you know why it is possible to store a numeric value in a character variables?


TeamCity pull requests :: netty [Build 151](https://garage.netty.io/teamcity/viewLog.html?buildId=992&buildTypeId=pulls_netty&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 4754, ignored: 21 Build time: 00:29:44


This branch contains a fairly efficient lookup of a language fallback. It is called when a translation cannot be found for the current language. In the current Zotonic, only `en` is considered as hardcoded fallback. The code changes make it possible to define a specific fallback per language, thus progressing one step towards region specific language settings like "fr-be” and "zh-tw”.

The major code change is that `#context.language` now contains a list of language code atoms, instead of a single language code. The first list item is the selected language. To keep other code working as the same as possible, `z_context:language/1` will still return the selected language.
Other code that directly read `Context#context.language` has been replaced with the `z_context:language/1` function call.
`z_context:set_language/2` accepts an atom and puts it in the list, but it will also take a list. That list will be set by `mod_translation` when a language has defined a fallback language.

Translation lookup takes one more lookup in case no translation is found. Fallback lookup is not recursive: only the fallback of the current language is explored. This also removes the need to program against infinite loops.
Currently only 1 fallback is considered, but it is built with a list of fallbacks in mind.
The ultimate fallback is `en` and that functions as it does today.


Thank you! Yes, my code (with alg3) is in sync with the Calender


A wonderful idea reusing the duplicate letters instead of creating duplicate variables. I suspect the usage of a good variable naming scheme was integral to making this observation.


merge


Typo!


It makes me think it will be cleaner in ModelPrinter class!


Alright, did the same with the patch list: 39e406c2d636140aad916aa970709233fb00fe60


United two methods in one.


Do your foresee any benefits or drawbacks from making a single statement of lines 21-30.


Thanks!


This does not return an integer value.
How could you modify this statement to return an integer.


This whole section doesn't need to be tabbed in as far as it is. The type pathing for it is absolute, but it is tabbed in as if it was relatively pathed.


Good job! :+1:

The only thing I noticed was the final result has a decimal point and should be an integer.


@geodotka proszę sprawdzić, czy się podoba. Poza tym chciałbym jeszcze, żeby na pierwszej stronie tych obrazków w miniaturach pojawiało się maksymalnie 3. Czy da się to zrobić jakoś prościej, niż przez oddzielny context[...] i dodanie do niego filtra w widokach?


Soft/hard tab inconsistency.


Ditto


LGTM


And I thought we keep it for sentimental reasons ;)

But seriously, if somebody still needs it, it can be still used
and doesn't hurt. Therefore, I kept it in.

But I'm also fine with removing it.


This line is awfully long. It would be better to break it up into multiple lines for readability, keeping your line length to 80 characters or less when possible.


So it'd be alright if I moved /obj/item/clothing/head/helmet/space/capspace/equipped(var/mob/living/carbon/human/user, var/slot) and what's under it one tab to the left? I thought I'd have to keep it this way because the /obj/item/clothing/head/helmet/space/capspace draws its 'flags' property from it


"the same as"

But I'm not sure why simple `ToMessage().ToString()` isn't enough.
If you don't change any behavior at all, what's the point of any changes then?

If some module called this function and expected that it can return either no tags or only `@time`, they have bad assumptions. Even if they learnt that sacred knowledge from implementation details (which they shouldn't), the implementation had a clear `// TODO support tags`.


what's some-file?


@tohojo: note that this also switched from open coded ins mod to $INSMOD, so this is an improvement of sorts...


Either we support post-OWIN WebParts, or we allow streaming. We should provide two types for this.


Well, the main problem is to figure out if/how we can get python to install the header files. You can simply look at the directories where pip puts the rebound files following a `pip install rebound` and there is no `.h` file...

    $ find . -name "*rebound*"
    ./venv/lib/python2.7/site-packages/librebound.so
    ./venv/lib/python2.7/site-packages/rebound
    ./venv/lib/python2.7/site-packages/rebound-2.7.0-py2.7.egg-info

Your solution works because you have all the rebound files on your computer, which wouldn't be the case for a pure python installation using pip.


Unhandled Fundamental Failure


that is useless here!


To create the contents in the demo, save two pieces with the user "user" and password "pass", with content "123" and "456", respectively. They should give the hashes in the sampleContent.html.
Then add the text from sampleContent.html as content. When getting this content, it will fill out the html with the referenced content.


Changing defaults is a really big deal in scalariform because it means upgrades incur a source change, usually across multiple branches, which interferes with feature merging. I'd encourage you to try and be 100% backwards compatible and to never change defaults. If someone wants formatting parity with another tool, then, well isn't that exactly what the preferences are there for?


This fixes the problems from last time, but there are still problems, which I see:
 * there must be a `_.escapeRegex` in `new RegExp(...)`
 * if a `link.text` occurs twice, each occurence is surrounded by two links, where one of them has the wrong ID (I currently don't have a laptop to test, but I think that is, what will happen)
 * this won't highlight links, where the ANSI color changes inside of the link, because `link.text` doesn't contain ANSI escape codes, but string does


87382760-VAVTerminalUnitSchedule-Issue4285 (mjwitte) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg)


Thanks!


Task 3
Adding support the calculation based on DFS algorithm

The main problem;
Distribution of tasks to multiple computers

The solution;
The server receives graph with edges, he use the function “Minimal Cliques” that calculate the edges from graph.
Then the server sends each time edge to one of the computers array (As calculation algorithm DFS, that sends vertex each time).
The computers calculates the length of maximal clique
From an edge that was sent to him.
The server receives databases from each of computer and updates the length of maximal clique as needed.
In the event, edge is not relevant and shorter then max clique, the server avoids sending.

A server is an instance of a computer program that accepts and responds to requests made by another program, known as a client.
The computers approached to server through a computer network to request information stored on the server.
Servers are used to manage network resources.
Some servers are committed to a specific task, often referred to as dedicated. As a result, there are a number of dedicated server categories, like print servers, file servers, network servers and database servers.
Because they are commonly used to deliver services that are required constantly, most servers are never turned off. Servers are commonly high-end computers setup to be fault tolerant.


не нравится что для переходов везде указываете путь. Оберните это в метод перехода на главную страницу, и только там 1 раз путь сохраните.


здесь tr[name="entry"] - уже не надо писать, вы ищите ячейку итак уже внутри элемента ряда


Really?  You're the only one working on this thing?  I coulda sworn there were like 12 people listed as contributors.  Harpy seems to be pretty active,  the others only seem to have a commit here and there I suppose.  I guess PVPGN's end of days is drawing near :(  I didnt realize just how dead it really was compared to when I last worked with it years ago.  I only recently began looking at it again.


The rangeIt can't be used to access the current partition with the min tuple after the pop_heap is called (inside the else if part). It still points to the front of the partitions vector which is not what we need. You have to get the 'back' element again.

        if (partitions.size() == 1 && rangeIt->first == rangeIt->second) {
            // The last partition is empty. Done.
            partitions.pop_back();
        } else if (partitions.size() > 1) {
            // There are more than one partition left. Remove the current top partition from the heap
            std::pop_heap(partitions.begin(), partitions.end(), reverseTupleRangeComp);
            // The rangeIt is invalidated at this point and can't be used to access the top partition
            tuple_range nextPartition = partitions.back();
            partitions.pop_back();
            if (nextPartition.first != nextPartition.second) {
                // The partition is not empty yet. Reinsert it back to the heap.
                partitions.push_back(nextPartition);
                std::push_heap(partitions.begin(), partitions.end(), reverseTupleRangeComp);
            }
        }

Changing the first line to be
tuple_range nextPartition = partitions.front();
to avoid having call partitions.back() also won't work because ++nextPartition.first increment the copy of the iterator and not the one that inside the partitions vector


ping @ryanjbaxter


@Tra-Vis Usually you just submit a pull request, and after we do the merge, you are free to fork a new copy off to your repository, which will contain a merge of your changes and our current source.


This commit has some modifications: https://github.com/kristophM/OpenTodo/commit/4dc40b871cb39840c51f1f573d99e73d1a202919


The official PvPGN development last ended in 2011: https://github.com/d1zzy/pvpgn

The last commit by Harpywar was on February 9th of this year.

Commits from everyone else are sparse and very random.


added


I would suggest you to re-rune the test with the following values, they should pass:

 { S(27, 7), S(43,12) }, // Knights
 { S(12, 2), S(17, 6) }  // Bishops

const Score ThreatenedByHangingPawn = S(38, 61);

const Score KingOnOne          = S( 0, 60);
const Score KingOnMany         = S( 1,122);
const Score RookOnPawn         = S( 7, 26);
const Score RookOnOpenFile     = S(46, 22);
const Score RookOnSemiOpenFile = S(20, 11);
const Score BishopPawns        = S(10, 13);
const Score MinorBehindPawn    = S(18,  0);
const Score TrappedRook        = S(84,  0);
const Score Unstoppable        = S( 0, 21);
const Score Hanging            = S(32, 28);
const Score PawnAttackThreat   = S(22, 21);


Sometimes the for loop variables are of type int, sometimes of std::size_t. Please check consistency.


I like this idea - but the patch lacks a changelog entry, or? Could you add one, please?


87382760-VAVTerminalUnitSchedule-Issue4285 (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1813 of 1813 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg)


87382760-VAVTerminalUnitSchedule-Issue4285 (mjwitte) - x86_64-MacOS-10.9-clang: OK (1807 of 1807 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg)


Careful with Naxxramas: the current trash mobs positions and movements are not the same than in Classic/TBC.

Regarding those NPCs, I remember making 88270 move because I saw it moving in an old movie from Vanilla. I don't say I'm 100% sure it is moving (my source is an old video, after all), but if I made it move, this is because I had good reasons to believe it was intended to. So I'm always open to acknowledge to my mistakes, but I'd like to know what make you think this NPC should stand idle, please.

Regarding 88272; it currently does not have WP movement, so I'm a bit puzzled by your fix.


Why do you do a merge step - you could rebase the code and add your patches to the head.


That's a funny half, Andrew.


yep! I agree that you do this shortly after here. Just something about the compiling


removed


ah ok. agreed!


Hi dude!
About "private void changeTitleBarColor() {", i found this issue at least on my slimlp mod build:
if i don't set a default app to open images, then i take a screenshot, then i open the screenshot heads up in floating mode, it gives a force close: http://pastebin.com/e3csE73p
So it seems we need a null check here:
final ComponentName cn = pm.getLaunchIntentForPackage(appPackageName).getComponent();

Also, when i enable heads up floating mode, apps run always in this mode also after a single tap on the heads up.

Thanks again for your amazing work!


Did you want to move this into the theme?


Actually you won't notice the second Problem, because both links point to the same line number since the text is the same. But if you look in the devtools, you'll see two links.


Thanks.  Thought i fixed that-guess not

On Sun, Aug 30, 2015 at 12:11 PM, Paul Bunyan <notifications@github.com>
wrote:

> Just so you know -- you pasted the URL twice into the href property.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Chad414/HOT_TeamWebSite/commit/fd615425d127d4645f1347eec05ac3a7800f59f0#commitcomment-12966064>
> .
>


What do you mean by that?


The list of tabs could be moved in the theme config file, I imagine that plugins or different sites could add new tabs


Muy bien señor Bauch, tiene aprobado diez :+1: cc @kiesslig95


That would be a nice future addition indeed! I do think though that page_content, page_seo and advanced are necessary basics. Perhaps we should make it extendable or overridable.


But we want translation marking already, thats important!

2015-08-30 19:54 GMT+02:00 Udayan Tandon <notifications@github.com>:

> So we decided that we are not adding translations now. The gui would have
> a translation framework setup later.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/coala-analyzer/coala-gui/commit/e91eab20fc7e5265d552ed78fbf25bc1c14d8039#commitcomment-12966409>
> .
>


@deads2k why do I have to specify both metadata.name and roleRef.view?


The visual studio is happily compiling the code. Remove the comment, and then it is good to be merged.
:fast_forward:


Mauriel and I talked through 3 cases:

1. Private - the resource is not discoverable and only those who have been given access can see/access the files.
2.  Discoverable - the resource is discoverable by any user. Users without access to the files can see the metadata on the landing page, but not the content files. Only users who have been given access can see/access the content files.
3.  Public - the resource is public, it is discoverable by any user, and any user can see/access the content files.

We need to look back at the documentation to make sure these are right, but if these are the cases, making a resource "private" would make it so that it is not discoverable.


4. 网页加载的时候，导航栏右上角的分享按钮暂时替换成菊花，等加载好后，再替换成原来的分享按钮(增加失败刷新按钮)


I'm not sure about the two additions here. See this video that I used as source when I cleaned up that wing of Naxxramas:
https://www.youtube.com/watch?v=IbRODBLeqk4
More in the whole commit: https://github.com/classicdb/database/commit/100e617a69d1d34c2d82c0b9f98400d3188309f2 and also in: https://github.com/classicdb/database/commit/44f59ba2a4596a545aea0de79fff9442e78085f9


I was refering to this video:
https://www.youtube.com/watch?v=IbRODBLeqk4
More in the whole commit: https://github.com/classicdb/database/commit/100e617a69d1d34c2d82c0b9f98400d3188309f2 and also in: https://github.com/classicdb/database/commit/44f59ba2a4596a545aea0de79fff9442e78085f9


Does this output an integer?


Because they charge u by the number of gets in s3, and if u specify a file
that's not there it'll just give you a permission denied error
On Aug 30, 2015 10:39 AM, "SorenOlegnowicz" <notifications@github.com>
wrote:

> Really? How would it cost them money?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/SorenOlegnowicz/zipdownloader/commit/2a3d3eb76ac37d12f1c3f5d64fc3edc43219ae1b#commitcomment-12965712>
> .
>


Nope. Now crashed obsidian and gravel could not be placed... But on the wiki it is said that they can be placed. @Yopu


This allows travis-ci to run the Rails 3 tests.


How could lines 22, 26, and 27 be combined into a single statement. Is there any benefit or drawback from combining them.


Yes here is a screen shot of what the program does.

[cid:615accb3-2dcf-4a19-a50a-593dd7df3d89]

Zachary T. Murphy
Computer and Information Technology Major
Westmoreland County Community College

Cell Phone: 724-516-7201

________________________________
From: Paul Scarrone <notifications@github.com>
Sent: Sunday, August 30, 2015 5:07 PM
To: ztmurphy21/Chapter2Assignment
Cc: Zachary Murphy
Subject: Re: [Chapter2Assignment] Code for Ch2Assignment (238f9bf)

Does this output an integer?

-
Reply to this email directly or view it on GitHub<https://github.com/ztmurphy21/Chapter2Assignment/commit/238f9bf7b72842b6ff6b9ae2e4d17f765bd9a191#commitcomment-12967118>.


87382760-VAVTerminalUnitSchedule-Issue4285 (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (759 of 759 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg) ![Coverage Badge](http://img.shields.io/badge/coverage%20status-9.37%25-green.svg)


And... the Rails 5 tests are passing again :)


This appears to be the output section of your code but this line is doing some processing.
You should always try and follow the section pattern:
`Declarations (and initialization)`
`Processing`
`Output`


Nice, like ABTs but a bit cooler!


what changed here?  The diff is inscrutable


The expected output should be an integer because all of our characters are integers.


and say f**k you to the net parasites out there :)


Brilliant this is what I need


Maybe:

f (pm !=null) {
final ComponentName cn = pm.getLaunchIntentForPackage(appPackageName).getComponent();
}

?


How could these variable names be changed to make the code more readable as well as inform us that some variables are redundant.


много пустых строк


Looks good :+1:


The added creatures can be seen around https://youtu.be/IbRODBLeqk4?t=149 to the very left. But there's actually another another patrol that the WotLK version is missing.


this.name = name;


can't find. don't know how to use it.


лишние импорты удалить


TeamCity pull requests :: netty [Build 152](https://garage.netty.io/teamcity/viewLog.html?buildId=993&buildTypeId=pulls_netty&guest=1) is now running


Is this better?

[cid:58daa9c8-c6b1-4b30-8637-bf9428af46b3]

Zachary T. Murphy
Computer and Information Technology Major
Westmoreland County Community College

Cell Phone: 724-516-7201

________________________________
From: Paul Scarrone <notifications@github.com>
Sent: Sunday, August 30, 2015 5:12 PM
To: ztmurphy21/Chapter2Assignment
Cc: Zachary Murphy
Subject: Re: [Chapter2Assignment] Code for Ch2Assignment (238f9bf)

Looks good [:+1:]

-
Reply to this email directly or view it on GitHub<https://github.com/ztmurphy21/Chapter2Assignment/commit/238f9bf7b72842b6ff6b9ae2e4d17f765bd9a191#commitcomment-12967147>.


Make that https://fonts.googleapis.com/icon?family=Material+Icons


Well about headsup thing it is because app stays in foreground floating launched. If you go to recents you will see that it is there. If you dont want that use the Close button on the title bar and it will kill the task nstead of minimize it


Doesn't this create unnecessary new variables?


I apologize your screenshots are not visible when responding via email


tnx for the hint ;)


Try with that null check


btw how did you know us?


Ok, here is a the screenshot in a word document. Let me know how it looks?

Thanks,

Zachary T. Murphy
Computer and Information Technology Major
Westmoreland County Community College

Cell Phone: 724-516-7201

________________________________
From: Paul Scarrone <notifications@github.com>
Sent: Sunday, August 30, 2015 5:15 PM
To: ztmurphy21/Chapter2Assignment
Cc: Zachary Murphy
Subject: Re: [Chapter2Assignment] Code for Ch2Assignment (238f9bf)

I apologize your screenshots are not visible when responding via email

-
Reply to this email directly or view it on GitHub<https://github.com/ztmurphy21/Chapter2Assignment/commit/238f9bf7b72842b6ff6b9ae2e4d17f765bd9a191#commitcomment-12967165>.


NO I NEEDED THAT


No what I mean is that attachments of any kind do not come through when you are replying via email.

If you follow the link to github you can attach your screenshots as well. https://github.com/ztmurphy21/Chapter2Assignment/commit/238f9bf7b72842b6ff6b9ae2e4d17f765bd9a191


Ok I uploaded the screenshot in Github. Sorry I didn't know that. Learned something new!
![untitled](https://cloud.githubusercontent.com/assets/13890535/9569504/6b898ce4-4f3b-11e5-9149-5f1f6c7e5c50.png)


Nice ! I'll put some review comments on some details ;)


Just copy `support/match.rb` and remove the `timestamps` parameter. Copying this hole file is overkill.


shut up


Did this bug have some precondition to happen?
I uploaded qutie some log pictures in the last weeks to gc.com but did not see any loose of description data while doing it.


@tjenkinson you're right, it doesn't need to be global. Actually we don't use this variable anymore (replaced with Log level), so it's safe to remove it,


maybe it's error number of issue: 689  -- >> 869 ?


Better not to use quotes around IDs (traits, flags, culture, religions etc.) , as vanilla doesn't do it, to be consistent.
It allows to easily differentiate IDs from other strings.


Should be an int


should use int


`trips` should be `abcd` as that is the desired route?


no


Trying now!
PS: i already knew ONE AxxionPop on xda but didn't notice you had a well working floating mode, an user pointed me to you ;D


Nice code, it's getting better. :D


You wrote mayor with a ```j``` on line 245 in sv_commands


Oh, nice job ! This code is clean and beautiful


Do you only want the output too show the number (integer) and nothing else?


So the concern here is the output

The word is 'perfect'.
The sum of the word perfect is  **745.0**.

I expect a better outcome would be **745**


I don't understand this commit actually - it was from some old code. I'll get rid of it.


song and dance is not copyright all this is Disney content you dont own shit mate


ps: trying this:

				if (pm !=null) {
				final ComponentName cn =                 pm.getLaunchIntentForPackage(appPackageName).getComponent();
				} else {
				final ComponentName cn = null;
				}


i own the cutscene code and the head joint fix


Well it doesn't apply here.


There seem to be a mix of spaces & tabs, so the indentation is off in some places.
Maker sure your text editor is configured to use tabs (like vanilla files)


yeah totally own a fix lol


Email is just hard you would think after all these years but no. Anyways thank you for uploading it directly


Hi Brady
tested at select patient -> edit vitals forms other notes field -> save


In case all events in a file use same namespace, no need to specify namespace = xxx multiple times.


I got the output to be 745, what do you think?


This change, breaks currently working code  for a Neo with the PIV application:
$ p11t -O
error: PKCS11 function C_OpenSession failed: rv = CKR_TOKEN_NOT_RECOGNIZED (0xe1)

Please add a patch to move the PIV driver before the openpgp drive in ctx.c line 109:
{ "PIV-II",     (void *(*)(void)) sc_get_piv_driver },
to line 75, just before:
{ "openpgp",    (void *(*)(void)) sc_get_openpgp_driver },

This will use the PIV application if its on the Neo, or try the openpgp driver.

Without this patch, opencard-openpgp.c  justs matches on the ATR, even though there is no openpgp application on the card. OpenSC then assumes card-openpgp is the driver. The card-piv is never called.

OpenSC does not handle very well how to select among multiple applications on the same card. Card driver are called in order and the first one to return matched becomes the driver.

If  individual user has only one application on the Neo, that should be the one selected.
If the user has two, it is not easy for a user to specify which card applicatrion should be used for
which program.  One way ti to have multi opensc.conf file. see below

The log with the change shows:
../../../src/src/libopensc/card.c:231:sc_connect_card: matched: OpenPGP card
This is only based on the ATR, not on any application on the card (default or not).

The card-piv
../../../src/src/libopensc/card-piv.c:2834:piv_match_card: called
 ../../../src/src/libopensc/card-piv.c:720:piv_find_aid: called
 ../../../src/src/libopensc/card-piv.c:683:piv_select_aid: called

sending a select to find the AID supported:
00 A4 04 00 09 A0 00 00 03 08 00 00 10 00 00
If it return 90 00

 ../../../src/src/libopensc/card-piv.c:735:piv_find_aid: found PIX
 ../../../src/src/libopensc/card-piv.c:749:piv_find_aid: returning with: 0 (Success)
 ../../../src/src/libopensc/card.c:231:sc_connect_card: matched: PIV-II  for multiple cards

Note that this is done duing the matching, and is not using the ATR of the card, but the fact that  a PIV application is on the card.

So if this patch patch is applied, the openpgp-card should be called after the card-piv.
This would only allow automatic selection of a Neo that has either openpgp or PIV, but says nothing about how to handle a Neo with two applications.

A user can override the order of drivers being called from opensc.conf something like:
card_drivers = openpgp internal;
or
card_drivers = PIV-II internal;

then setting export OPENSC_CONF=/path/to/opensc.conf.xxx


Just removed this in a2a6d4288fbc71b926ee860b9238ba904f7958e0. Thanks for pointing this out, @tjenkinson.


oops, fixed.


ctxthmap->nvalues should be updated somewhere


This API does not let stack contexts.

With previous API you could do:

int prev_ctx = SWITCH_CONTEXT_TO(lib_context);
do_lib_ctx_ops();
SWITCH_CONTEXT_TO(prev_ctx);

With ENTER_APP/LIB_CONTEXT API you must know surrounding context to not mess up state.


Make sure the file has .txt extension, otherwise it won't be loaded by the game.